RESOLVED FIXED 83109
Followup to "Call incrementStatsCounter directly"
https://bugs.webkit.org/show_bug.cgi?id=83109
Summary Followup to "Call incrementStatsCounter directly"
Mark Pilgrim (Google)
Reported 2012-04-03 19:54:36 PDT
Followup to "Call incrementStatsCounter directly"
Attachments
Patch (1.17 KB, patch)
2012-04-03 19:55 PDT, Mark Pilgrim (Google)
no flags
Work-in-Progress Patch, please ignore (6.99 KB, patch)
2012-04-04 14:23 PDT, Mark Pilgrim (Google)
no flags
Patch (21.96 KB, patch)
2012-04-18 07:05 PDT, Mark Pilgrim (Google)
no flags
Patch (21.98 KB, patch)
2012-04-18 08:14 PDT, Mark Pilgrim (Google)
no flags
Patch (21.96 KB, patch)
2012-04-18 08:51 PDT, Mark Pilgrim (Google)
no flags
Mark Pilgrim (Google)
Comment 1 2012-04-03 19:55:28 PDT
Kentaro Hara
Comment 2 2012-04-03 21:00:22 PDT
Comment on attachment 135491 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=135491&action=review > Source/WebCore/bindings/v8/V8Proxy.h:49 > +#define INC_STATS(name) PlatformSupport::incrementStatsCounter(name) Sorry, I am a bit confused. You removed PlatformSupport::incrementStatsCounter in r113034, and thus PlatformSupport::incrementStatsCounter no longer exists... doesn't it?
Adam Barth
Comment 3 2012-04-03 21:25:22 PDT
Comment on attachment 135491 [details] Patch Mark, you want to introduce an abstraction like <http://trac.webkit.org/browser/trunk/Source/WebCore/platform/HistogramSupport.h>, perhaps called StatsCounter.h. We'll implement StatsCounterChromium.cpp via WebKit::Platform and other ports can implement StatsCounterWhatever.cpp however they like (if they want stats counting.
Mark Pilgrim (Google)
Comment 4 2012-04-04 14:23:30 PDT
Created attachment 135681 [details] Work-in-Progress Patch, please ignore
Mark Pilgrim (Google)
Comment 5 2012-04-04 14:24:04 PDT
Comment on attachment 135681 [details] Work-in-Progress Patch, please ignore Not ready for review.
Mark Pilgrim (Google)
Comment 6 2012-04-18 07:05:49 PDT
Build Bot
Comment 7 2012-04-18 07:31:03 PDT
Mark Pilgrim (Google)
Comment 8 2012-04-18 08:14:57 PDT
Kentaro Hara
Comment 9 2012-04-18 08:43:40 PDT
Comment on attachment 137693 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=137693&action=review OK > Source/WebCore/platform/StatsCounter.h:38 > + static void incrementStatsCounter(const char* counterName); Nit: 'counterName' is not necessary.
Kentaro Hara
Comment 10 2012-04-18 08:48:17 PDT
Comment on attachment 137693 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=137693&action=review >> Source/WebCore/platform/StatsCounter.h:38 >> + static void incrementStatsCounter(const char* counterName); > > Nit: 'counterName' is not necessary. Would you please remove this before commit?
Mark Pilgrim (Google)
Comment 11 2012-04-18 08:51:10 PDT
WebKit Review Bot
Comment 12 2012-04-18 09:29:39 PDT
Comment on attachment 137703 [details] Patch Clearing flags on attachment: 137703 Committed r114515: <http://trac.webkit.org/changeset/114515>
WebKit Review Bot
Comment 13 2012-04-18 09:29:45 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.