Summary: | Don't force layout when querying a fixed or non-box margin/padding property | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||
Component: | CSS | Assignee: | Ryosuke Niwa <rniwa> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | barraclough, commit-queue, eric, esprehn+autocc, glenn, hyatt, kling, koivisto, macpherson, menard, mjs, simon.fraser, tonyg | ||||
Priority: | P2 | Keywords: | BlinkMergeCandidate | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | 118930, 118936 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Ryosuke Niwa
2013-06-25 23:07:48 PDT
Don't merge verbatim. isLayoutDependent should not take a PassRefPtr<RenderStyle> style. Also there should be an inline helper function for all these renderer && renderer->isBox() && (!style || !style->paddingBottom().isFixed()); Also see https://chromium.googlesource.com/chromium/blink/+/ff234b1593b2b493d47f38f687d09a87bc42c9eb. Also see the bug 118618. Created attachment 207433 [details]
Merges the patch
There were two follow-ups you might be interested in: https://codereview.chromium.org/18298016/ https://codereview.chromium.org/19272007/ (In reply to comment #5) > There were two follow-ups you might be interested in: > https://codereview.chromium.org/18298016/ > https://codereview.chromium.org/19272007/ Thanks but those are follow ups for the patch to be merged in the bug 118618, right? Comment on attachment 207433 [details]
Merges the patch
r=me
(In reply to comment #6) > (In reply to comment #5) > > There were two follow-ups you might be interested in: > > https://codereview.chromium.org/18298016/ > > https://codereview.chromium.org/19272007/ > > Thanks but those are follow ups for the patch to be merged in the bug 118618, right? You are right. Sorry, I got the bugs confused. Comment on attachment 207433 [details] Merges the patch Clearing flags on attachment: 207433 Committed r153347: <http://trac.webkit.org/changeset/153347> All reviewed patches have been landed. Closing bug. |