RESOLVED FIXED 83023
Call incrementStatsCounter directly 
https://bugs.webkit.org/show_bug.cgi?id=83023
Summary Call incrementStatsCounter directly
Mark Pilgrim (Google)
Reported 2012-04-03 06:09:30 PDT
Call incrementStatsCounter directly
Attachments
Patch (3.77 KB, patch)
2012-04-03 06:10 PDT, Mark Pilgrim (Google) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Pilgrim (Google)
Comment 1 2012-04-03 06:10:12 PDT
Created attachment 135319 [details] Patch
Kentaro Hara
Comment 2 2012-04-03 06:14:47 PDT
Comment on attachment 135319 [details] Patch OK!
WebKit Review Bot
Comment 3 2012-04-03 08:03:44 PDT
Comment on attachment 135319 [details] Patch Clearing flags on attachment: 135319 Committed r113034: <http://trac.webkit.org/changeset/113034>
WebKit Review Bot
Comment 4 2012-04-03 08:03:49 PDT
All reviewed patches have been landed. Closing bug.
Adam Barth
Comment 5 2012-04-03 08:45:25 PDT
Comment on attachment 135319 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=135319&action=review > Source/WebCore/bindings/v8/V8Proxy.h:49 > -#define INC_STATS(name) PlatformSupport::incrementStatsCounter(name) > +#define INC_STATS(name) webkitPlatformSupport()->incrementStatsCounter(name) Does this compile with ENABLE_DOM_STATS_COUNTERS enabled? webkitPlatformSupport shouldn't be visible outside the WebKit layer, but this macro is used in WebCore.
Kentaro Hara
Comment 6 2012-04-03 08:50:26 PDT
Comment on attachment 135319 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=135319&action=review >> Source/WebCore/bindings/v8/V8Proxy.h:49 >> +#define INC_STATS(name) webkitPlatformSupport()->incrementStatsCounter(name) > > Does this compile with ENABLE_DOM_STATS_COUNTERS enabled? webkitPlatformSupport shouldn't be visible outside the WebKit layer, but this macro is used in WebCore. Ah, good point, the indirection call is necessary. May I roll the patch out?
Adam Barth
Comment 7 2012-04-03 10:02:30 PDT
> Ah, good point, the indirection call is necessary. May I roll the patch out? I don't think anyone commonly builds this configuration, so we might want to wait for Mark to fix it.
Mark Pilgrim (Google)
Comment 8 2012-04-03 19:55:40 PDT
(In reply to comment #7) > > Ah, good point, the indirection call is necessary. May I roll the patch out? > > I don't think anyone commonly builds this configuration, so we might want to wait for Mark to fix it. Fix is bug 83109.
Note You need to log in before you can comment on or make changes to this bug.