Bug 83112

Summary: Call histogramCustomCounts directly
Product: WebKit Reporter: Mark Pilgrim (Google) <pilgrim>
Component: New BugsAssignee: Mark Pilgrim (Google) <pilgrim>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, cc-bugs, fishd, jamesr, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 82948    
Attachments:
Description Flags
Patch
none
Patch none

Description Mark Pilgrim (Google) 2012-04-03 20:22:47 PDT 
Call histogramCustomCounts directly
Comment 1 Mark Pilgrim (Google) 2012-04-03 20:23:17 PDT 
Created attachment 135497 [details]
Patch
Comment 2 James Robinson 2012-04-03 20:41:24 PDT 
Comment on attachment 135497 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=135497&action=review

I'd like Adam to check that this is the intended use of WebKit::Platform, but if so then this looks all right

> Source/WebKit/chromium/src/painting/PaintAggregator.cpp:36
> +#include <public/Platform.h>

this seems spurious
Comment 3 Adam Barth 2012-04-03 21:23:24 PDT 
Comment on attachment 135497 [details]
Patch

Yep.  Looks great.  Thanks.
Comment 4 Mark Pilgrim (Google) 2012-04-04 06:46:06 PDT 
Created attachment 135581 [details]
Patch
Comment 5 Mark Pilgrim (Google) 2012-04-04 06:47:05 PDT 
Removed spurious include in PaintAggregator.cpp.
Comment 6 WebKit Review Bot 2012-04-04 12:46:48 PDT 
Comment on attachment 135581 [details]
Patch

Clearing flags on attachment: 135581

Committed r113227: <http://trac.webkit.org/changeset/113227>
Comment 7 WebKit Review Bot 2012-04-04 12:46:53 PDT 
All reviewed patches have been landed.  Closing bug.