Summary: | CSS3 calc: rewrite margin test to use getComputedStyle | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mike Lawther <mikelawther> | ||||||||
Component: | New Bugs | Assignee: | Mike Lawther <mikelawther> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | tkent, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 16662 | ||||||||||
Attachments: |
|
Description
Mike Lawther
2012-03-05 22:18:57 PST
Created attachment 130292 [details]
Patch
Created attachment 130293 [details]
Patch
Comment on attachment 130293 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=130293&action=review > LayoutTests/css3/calc/margin-expected.txt:2 > +PASS getComputedStyle(document.getElementById("simple-all"), null).marginLeft is "25px" > +PASS getComputedStyle(document.getElementById("simple-all"), null).marginTop is "25px" nit: The result looks too busy. We can simplify it by adding functions. e.g. computedMarginLeft("simple-all") computedMarginTop("simple-all") ... > nit: The result looks too busy. We can simplify it by adding functions. e.g.
> computedMarginLeft("simple-all")
> computedMarginTop("simple-all")
Good idea - done.
Created attachment 130297 [details]
Patch for landing
Comment on attachment 130297 [details] Patch for landing Clearing flags on attachment: 130297 Committed r109874: <http://trac.webkit.org/changeset/109874> All reviewed patches have been landed. Closing bug. |