WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
Formatted Diff
Diff
Work-in-Progress Patch, please ignore
(6.99 KB, patch)
2012-04-04 14:23 PDT
,
Mark Pilgrim (Google)
no flags
Details
Formatted Diff
Diff
Patch
(21.96 KB, patch)
2012-04-18 07:05 PDT
,
Mark Pilgrim (Google)
no flags
Details
Formatted Diff
Diff
Patch
(21.98 KB, patch)
2012-04-18 08:14 PDT
,
Mark Pilgrim (Google)
no flags
Details
Formatted Diff
Diff
Patch
(21.96 KB, patch)
2012-04-18 08:51 PDT
,
Mark Pilgrim (Google)
no flags
Details
Formatted Diff
Diff
Show Obsolete
(4)
View All
Add attachment
proposed patch, testcase, etc.
Mark Pilgrim (Google)
Comment 1
2012-04-03 19:55:28 PDT
Created
attachment 135491
[details]
Patch
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
Created
attachment 137680
[details]
Patch
Build Bot
Comment 7
2012-04-18 07:31:03 PDT
Comment on
attachment 137680
[details]
Patch
Attachment 137680
[details]
did not pass mac-ews (mac): Output:
http://queues.webkit.org/results/12430074
Mark Pilgrim (Google)
Comment 8
2012-04-18 08:14:57 PDT
Created
attachment 137693
[details]
Patch
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
Created
attachment 137703
[details]
Patch
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.
Top of Page
Format For Printing
XML
Clone This Bug