Bug 119560 - Don't remove contents layer from its parent unless necessary
Summary: Don't remove contents layer from its parent unless necessary
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: Dean Jackson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-08-07 15:07 PDT by Dean Jackson
Modified: 2013-08-07 15:15 PDT (History)
5 users (show)

See Also:

Attachments
Patch (1.88 KB, patch)
2013-08-07 15:09 PDT, Dean Jackson 		simon.fraser: review+
Details | Formatted Diff | Diff
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 Dean Jackson 2013-08-07 15:07:29 PDT 
RenderLayerBacking::updateInternalHierarchy() is called in some instances where there doesn't need to be a complete reparenting of the the m_graphicsLayer. For example, on OS X when the user changes the scrollbar styles to/from always visible. We don't need to always remove the graphics layer from its parent - any reparenting does that automatically.

Unfortunately we can't trigger this situation in an automated test.

<rdar://problem/14290395>
Comment 1 Dean Jackson 2013-08-07 15:09:18 PDT 
Created attachment 208302 [details]
Patch
Comment 2 Dean Jackson 2013-08-07 15:15:12 PDT 
Committed r153805: <http://trac.webkit.org/changeset/153805>