[webkit-reviews] review requested: [Bug 42093] Change handling of -khtml- and -apple- vendor-prefixes : [Attachment 61501] Added 3 test-cases

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 14 04:35:08 PDT 2010


Peter Beverloo <peter at lvp-media.com> has asked	for review:
Bug 42093: Change handling of -khtml- and -apple- vendor-prefixes
https://bugs.webkit.org/show_bug.cgi?id=42093

Attachment 61501: Added 3 test-cases
https://bugs.webkit.org/attachment.cgi?id=61501&action=review

------- Additional Comments from Peter Beverloo <peter at lvp-media.com>
The hasPrefix function does not touch the length property, therefore I believe
the code to be correct.

I have added three testcases:
 - Testing (limited) support for the -apple vendor prefix.
 - Testing whether the -khtml prefix actually got removed.
 - Testing whether the -webkit-opacity legacy-property got removed.

One test-cases got updated, "apple-prefix.html" which tests -apple-line-clamp
and relied on -khtml.
One test-case got removed, "legacy-opacity-styles.html", now tested by the
case which tests whether the property got removed.


More information about the webkit-reviews mailing list