Bug 66896

Summary: Remove LayoutStateDisabler instances from RenderLayer
Product: WebKit Reporter: Julien Chaffraix <jchaffraix>
Component: Layout and RenderingAssignee: Julien Chaffraix <jchaffraix>
Status: RESOLVED FIXED    
Severity: Minor CC: simon.fraser, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Remove 2 instances of LayoutStateDisabler that are not needed anymore.
none
Version 2: Remove a bad wording (layers' are not sparse or at least this is not guaranteed).
none
Patch for landing none

Julien Chaffraix
Reported 2011-08-24 14:48:29 PDT
RenderLayer::scrollTo does not call updateLayerPositions anymore so they can be safely removed.
Attachments
Remove 2 instances of LayoutStateDisabler that are not needed anymore. (3.81 KB, patch)
2011-08-24 15:03 PDT, Julien Chaffraix 		no flags
DetailsFormatted DiffDiff
Version 2: Remove a bad wording (layers' are not sparse or at least this is not guaranteed). (3.84 KB, patch)
2011-08-24 15:25 PDT, Julien Chaffraix 		no flags
DetailsFormatted DiffDiff
Patch for landing (3.86 KB, patch)
2011-09-07 14:51 PDT, Julien Chaffraix 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Julien Chaffraix
Comment 1 2011-08-24 15:03:01 PDT
Created attachment 105080 [details] Remove 2 instances of LayoutStateDisabler that are not needed anymore.
Julien Chaffraix
Comment 2 2011-08-24 15:25:00 PDT
Created attachment 105082 [details] Version 2: Remove a bad wording (layers' are not sparse or at least this is not guaranteed).
Simon Fraser (smfr)
Comment 3 2011-09-07 14:11:07 PDT
Comment on attachment 105082 [details] Version 2: Remove a bad wording (layers' are not sparse or at least this is not guaranteed). View in context: https://bugs.webkit.org/attachment.cgi?id=105082&action=review > Source/WebCore/ChangeLog:16 > + that was not acurate anymore. Also explained what is wrong with accurate
Julien Chaffraix
Comment 4 2011-09-07 14:51:35 PDT
Created attachment 106647 [details] Patch for landing
WebKit Review Bot
Comment 5 2011-09-08 11:53:40 PDT
Comment on attachment 106647 [details] Patch for landing Clearing flags on attachment: 106647 Committed r94777: <http://trac.webkit.org/changeset/94777>
WebKit Review Bot
Comment 6 2011-09-08 11:53:44 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.