Bug 77137 - [chromium] Remove dead and unnecessary code related to LayerChromium::cleanupResources
Summary: [chromium] Remove dead and unnecessary code related to LayerChromium::cleanup...
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: Alok Priyadarshi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-26 13:48 PST by Alok Priyadarshi
Modified: 2012-01-26 15:14 PST (History)
5 users (show)

See Also:

Attachments
proposed patch (2.74 KB, patch)
2012-01-26 13:53 PST, Alok Priyadarshi 		no flags 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 Alok Priyadarshi 2012-01-26 13:48:42 PST 
LayerChromium::cleanupResourcesRecursive() seems dead.
Comment 1 Alok Priyadarshi 2012-01-26 13:53:04 PST 
Created attachment 124174 [details]
proposed patch
Comment 2 Alok Priyadarshi 2012-01-26 13:54:19 PST 
Comment on attachment 124174 [details]
proposed patch

View in context: https://bugs.webkit.org/attachment.cgi?id=124174&action=review

> Source/WebCore/platform/graphics/chromium/TiledLayerChromium.cpp:-181
> -    if (layerTreeHost())

This is unnecessary. LayerChromium::setLayerTreeHost() already does this.
Comment 3 James Robinson 2012-01-26 13:56:49 PST 
Comment on attachment 124174 [details]
proposed patch

R=me, this has been dead a while. Thanks for removing it.
Comment 4 WebKit Review Bot 2012-01-26 15:13:59 PST 
Comment on attachment 124174 [details]
proposed patch

Clearing flags on attachment: 124174

Committed r106052: <http://trac.webkit.org/changeset/106052>
Comment 5 WebKit Review Bot 2012-01-26 15:14:03 PST 
All reviewed patches have been landed.  Closing bug.