Bug 97108 - [CoordinatedGraphics] Don't reset m_shouldSyncFrame in flushPendingLayerChanges
Summary: [CoordinatedGraphics] Don't reset m_shouldSyncFrame in flushPendingLayerChanges
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Balazs Kelemen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-19 06:51 PDT by Balazs Kelemen
Modified: 2012-09-20 04:13 PDT (History)
2 users (show)

See Also:

Attachments
Patch (1.78 KB, patch)
2012-09-19 06:54 PDT, Balazs Kelemen 		no flags Details | Formatted Diff | Diff
Patch (1.91 KB, patch)
2012-09-20 02:27 PDT, Balazs Kelemen 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Balazs Kelemen 2012-09-19 06:51:22 PDT 
We should consider the changes resulted in syncLayerState, syncLayerChildren, etc., that's why those methods set it.
Comment 1 Balazs Kelemen 2012-09-19 06:54:50 PDT 
Created attachment 164733 [details]
Patch
Comment 2 Noam Rosenthal 2012-09-19 07:00:05 PDT 
Comment on attachment 164733 [details]
Patch

Please say in the changelog why this is important (e.g. changes can happen during layout, like deleting a layer)
Comment 3 Balazs Kelemen 2012-09-20 02:27:37 PDT 
Created attachment 164866 [details]
Patch
Comment 4 Balazs Kelemen 2012-09-20 04:13:46 PDT 
Comment on attachment 164866 [details]
Patch

Clearing flags on attachment: 164866

Committed r129116: <http://trac.webkit.org/changeset/129116>
Comment 5 Balazs Kelemen 2012-09-20 04:13:50 PDT 
All reviewed patches have been landed.  Closing bug.