Bug 134318

Summary: [Win] Expose Cache Information to WinLauncher
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: Tools / TestsAssignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, bunhere, cdumez, commit-queue, dino, gyuyoung.kim, sergio, thorton
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch dino: review+

Description Brent Fulgham 2014-06-25 17:14:32 PDT 
It would be useful to be able to see information about memory use and cache status of various objects in WebKit.

This patch exposes the contents of our IWebCache and IWebCoreStatistics interfaces to the WinLauncher UI to make it easier to see this information.
Comment 1 Brent Fulgham 2014-06-25 17:43:58 PDT 
Created attachment 233858 [details]
Patch
Comment 2 Brent Fulgham 2014-06-26 09:15:21 PDT 
Created attachment 233906 [details]
Patch
Comment 3 Brent Fulgham 2014-06-26 09:18:58 PDT 
Created attachment 233907 [details]
Patch
Comment 4 Brent Fulgham 2014-06-26 09:20:33 PDT 
EWS bots are being difficult!
Comment 5 Brent Fulgham 2014-06-26 09:21:28 PDT 
Created attachment 233908 [details]
Patch
Comment 6 Brent Fulgham 2014-06-26 09:56:43 PDT 
Created attachment 233910 [details]
Patch
Comment 7 Brent Fulgham 2014-06-26 09:57:34 PDT 
Looks like EOL-foo was confusing the bots. I landed a separate fix for the multiple EOL-styles in WinLauncher.h. Hopefully that will allow this patch to be processed successfully.
Comment 8 Dean Jackson 2014-06-26 10:09:07 PDT 
Comment on attachment 233910 [details]
Patch

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

rs=me

> Source/WebKit/win/WebCache.cpp:113
> +    // Counts

Nit: Missing period here and on the other new comments.
Comment 9 Brent Fulgham 2014-06-26 10:22:37 PDT 
Committed r170483: <http://trac.webkit.org/changeset/170483>