Bug 84458 - [chromium] Remove special case for recreating layers during sync
Summary: [chromium] Remove special case for recreating layers during sync
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dana Jansens
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-20 09:03 PDT by Dana Jansens
Modified: 2012-04-20 17:28 PDT (History)
9 users (show)

See Also:


Attachments
Patch (5.39 KB, patch)
2012-04-20 09:05 PDT, Dana Jansens
no flags Details | Formatted Diff | Diff
Patch (4.37 KB, patch)
2012-04-20 14:26 PDT, Dana Jansens
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dana Jansens 2012-04-20 09:03:28 PDT
[chromium] Move nonFastScrollableRegion special case code to pushPropertiesTo with generic mechanism other properties may use
Comment 1 Dana Jansens 2012-04-20 09:05:42 PDT
Created attachment 138103 [details]
Patch
Comment 2 James Robinson 2012-04-20 10:39:16 PDT
Comment on attachment 138103 [details]
Patch

This doesn't seem worth abstracting - I don't think this is going to be a general mechanism.  I think we can just delete the TreeSynchronizer code completely now that we don't delete the impl thread
Comment 3 Dana Jansens 2012-04-20 14:26:08 PDT
Created attachment 138165 [details]
Patch

ok, makes sense.
Comment 4 James Robinson 2012-04-20 15:59:35 PDT
Comment on attachment 138165 [details]
Patch

Yay
Comment 5 WebKit Review Bot 2012-04-20 17:28:05 PDT
Comment on attachment 138165 [details]
Patch

Clearing flags on attachment: 138165

Committed r114808: <http://trac.webkit.org/changeset/114808>
Comment 6 WebKit Review Bot 2012-04-20 17:28:09 PDT
All reviewed patches have been landed.  Closing bug.