Summary: | CSS3 calc: unprefix implementation | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mike Lawther <mikelawther> | ||||
Component: | CSS | Assignee: | Mike Lawther <mikelawther> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | 7raivis, cmarcelo, ericbidelman, kling, koivisto, macpherson, mathias, menard, ojan.autocc, ojan, paulirish, rniwa, shikolay, simon.fraser, syoichi, tony, webkit.bugzilla, webkit.review.bot | ||||
Priority: | P2 | Keywords: | WebExposed | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 16662 | ||||||
Attachments: |
|
Description
Mike Lawther
2012-07-22 16:44:11 PDT
I've tested -webkit-calc() against both the IE and Mozilla test suites. IE : PASS Moz : PASS everything except: - text-indent-intrinsic-1.html : this is a problem in WebKit's text-indent intrinsic - width-table-auto-1.html, width-table-fixed-1.html : both table related, and per spec it is acceptable to ignore calc() in tables. The spec (http://www.w3.org/TR/css3-values/) is now at Candidate Rec. With the spec at CR, two shipping unprefixed implementations, and broad compat between ours and theirs, I reckon it's OK to go ahead with the unprefixing. Just awaiting for unprefix. Created attachment 183701 [details]
Patch
Testing-wise, I've replaced all -webkit-calc() in the tests with calc(), as the unprefixed version is what we'll support from now on. I've kept a copy of the most basic test using -webkit-calc() to ensure our support for it doesn't completely break. This seemed a sane approach - keeping an unprefixed copy of the entire test directory seemed like overkill. (In reply to comment #1) > Moz : PASS everything except: > - text-indent-intrinsic-1.html : this is a problem in WebKit's text-indent intrinsic > - width-table-auto-1.html, width-table-fixed-1.html : both table related, and per spec it is acceptable to ignore calc() in tables. Please file bugs to track these failures. Did you try the Mozilla tests after fixes are landed in https://bugzilla.mozilla.org/show_bug.cgi?id=826582 ? Text indent bug filed as https://bugs.webkit.org/show_bug.cgi?id=107416. wrt filing bugs about the table differences - both our impls are to spec. All the bug would say is 'this behaviour is different'? I did not run the tests after Moz upstreamed them to the W3C. I can do this and report back. Comment on attachment 183701 [details]
Patch
<3
(In reply to comment #7) > wrt filing bugs about the table differences - both our impls are to spec. All the bug would say is 'this behaviour is different'? That would be great. It'll helps us document the issue at least. If anything, we should check if there's any compatibility concerns to pick one behavior over another. Comment on attachment 183701 [details] Patch Clearing flags on attachment: 183701 Committed r140300: <http://trac.webkit.org/changeset/140300> All reviewed patches have been landed. Closing bug. As requested, I tested Mozilla's updated tests from http://mxr.mozilla.org/mozilla-central/source/layout/reftests/w3c-css/submitted/values3/, and the results were the same as I reported in comment #1. Aside - now that we are both unprefixed, I was able to do this testing without modifying any test source code - yay for compat! (In reply to comment #12) > As requested, I tested Mozilla's updated tests from http://mxr.mozilla.org/mozilla-central/source/layout/reftests/w3c-css/submitted/values3/, and the results were the same as I reported in comment #1. > > Aside - now that we are both unprefixed, I was able to do this testing without modifying any test source code - yay for compat! Thanks for the follow up! Table differences filed as https://bugs.webkit.org/show_bug.cgi?id=107482 |