Summary: | Only throttle DOM timers if they change CSS properties that cannot cause non-descendant elements to become visible | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||
Component: | CSS | Assignee: | Chris Dumez <cdumez> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | barraclough, commit-queue, dbates, kling, koivisto, simon.fraser | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Chris Dumez
2015-01-06 13:02:42 PST
Created attachment 244093 [details]
Patch
Comment on attachment 244093 [details] Patch Clearing flags on attachment: 244093 Committed r178014: <http://trac.webkit.org/changeset/178014> All reviewed patches have been landed. Closing bug. Comment on attachment 244093 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=244093&action=review > LayoutTests/fast/dom/timer-height-change-no-throttling.html:31 > + setTimeout(scrollElementIntoView, 0); Did you mean to include this line? If so, scrollElementIntoView needs to be defined. Comment on attachment 244093 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=244093&action=review >> LayoutTests/fast/dom/timer-height-change-no-throttling.html:31 >> + setTimeout(scrollElementIntoView, 0); > > Did you mean to include this line? If so, scrollElementIntoView needs to be defined. No, this line needs to go away. Reopening to attach new patch. Created attachment 244563 [details]
Patch
Comment on attachment 244563 [details] Patch Clearing flags on attachment: 244563 Committed r178403: <http://trac.webkit.org/changeset/178403> All reviewed patches have been landed. Closing bug. |