Bug 77137

Summary:

[chromium] Remove dead and unnecessary code related to LayerChromium::cleanupResources

Product:

WebKit

Reporter:

Alok Priyadarshi <alokp>

Component:

Layout and Rendering

Assignee:

Alok Priyadarshi <alokp>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

cc-bugs, jamesr, nduca, vangelis, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
proposed patch	none

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.