Bug 73060 - Cleanup #if usage in V8GCController
: Cleanup #if usage in V8GCController
Status: RESOLVED FIXED
Product: WebKit
Classification: Unclassified
Component: WebCore Misc.
: 528+ (Nightly build)
: Unspecified Unspecified
: P2 Normal
Assigned To: Rafael Weinstein
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-11-23 16:33 PST by Rafael Weinstein
Modified: 2011-11-23 19:24 PST (History)
3 users (show)

See Also:


Attachments
Patch (1.69 KB, patch)
2011-11-23 16:35 PST, Rafael Weinstein
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rafael Weinstein 2011-11-23 16:33:36 PST
Multiple patches have left checkMemoryUsage in a silly state
Comment 1 Rafael Weinstein 2011-11-23 16:35:10 PST
Created attachment 116452 [details]
Patch
Comment 2 Ojan Vafai 2011-11-23 16:38:20 PST
Comment on attachment 116452 [details]
Patch

This is fine, but I doubt we need the #if at all. AFAIK, V8GCController is only included by Chromium and QT anyways.
Comment 3 WebKit Review Bot 2011-11-23 19:24:41 PST
Comment on attachment 116452 [details]
Patch

Clearing flags on attachment: 116452

Committed r101117: <http://trac.webkit.org/changeset/101117>
Comment 4 WebKit Review Bot 2011-11-23 19:24:45 PST
All reviewed patches have been landed.  Closing bug.