Summary: | [chromium] Remove old cleanupResources() code | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dana Jansens <danakj> | ||||
Component: | New Bugs | Assignee: | Dana Jansens <danakj> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | backer, cc-bugs, enne, jamesr, piman, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Dana Jansens
2012-03-05 08:06:11 PST
Created attachment 130139 [details]
Patch
cleanupResources() is called from destructors, primarily to call cleanupResources on the layer's RenderSurface. That surface is held by an OwnPtr, so its destructor will be called anyways. And RS::cleanupResources is used to clear OwnPtrs within the surface. Basically, this function is redundant with destructors now, though I imagine it served some larger purpose in the past, which has been solved with OwnPtrs now. Comment on attachment 130139 [details]
Patch
Nice cleanup! :)
Comment on attachment 130139 [details] Patch Clearing flags on attachment: 130139 Committed r109781: <http://trac.webkit.org/changeset/109781> All reviewed patches have been landed. Closing bug. |