Bug 185853

Summary: Add notifyutil callbacks to dump the memory cache, and the list of live Pages and Documents
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, commit-queue, dbates, dewei_zhu, ews-watchlist, ggaren, japhet, keith_miller, rniwa, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
keith_miller: review+
Patch
ews-watchlist: commit-queue-
Archive of layout-test-results from ews107 for mac-sierra-wk2
none
Patch none

Description Simon Fraser (smfr) 2018-05-21 17:18:45 PDT
Add notifyutil callbacks to dump the memory cache, and the list of live Pages and Documents
Comment 1 Simon Fraser (smfr) 2018-05-21 17:20:10 PDT
Created attachment 340930 [details]
Patch
Comment 2 EWS Watchlist 2018-05-21 17:23:20 PDT
Attachment 340930 [details] did not pass style-queue:


ERROR: Source/WebCore/page/mac/PageMac.mm:81:  One line control clauses should not use braces.  [whitespace/braces] [4]
Total errors found: 1 in 9 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Keith Miller 2018-05-21 17:26:41 PDT
Comment on attachment 340930 [details]
Patch

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

r=me. Seems reasonable.

> Source/WebCore/ChangeLog:11
> +        Document pointer address and URL.

Maybe it's worth putting a brief example of how to use notifyutil on MacOS? For those of us unsure of exactly how to do it... e.g. me >.>
Comment 4 Simon Fraser (smfr) 2018-05-22 14:38:37 PDT
Created attachment 341029 [details]
Patch
Comment 5 EWS Watchlist 2018-05-22 14:40:17 PDT
Attachment 341029 [details] did not pass style-queue:


ERROR: Source/WebCore/page/mac/PageMac.mm:81:  One line control clauses should not use braces.  [whitespace/braces] [4]
Total errors found: 1 in 9 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 EWS Watchlist 2018-05-22 16:11:39 PDT
Comment on attachment 341029 [details]
Patch

Attachment 341029 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/7770041

New failing tests:
imported/w3c/web-platform-tests/css/css-text/text-transform/text-transform-capitalize-020.html
Comment 7 EWS Watchlist 2018-05-22 16:11:41 PDT
Created attachment 341043 [details]
Archive of layout-test-results from ews107 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 8 Simon Fraser (smfr) 2018-05-22 16:14:54 PDT
Created attachment 341044 [details]
Patch
Comment 9 EWS Watchlist 2018-05-22 16:23:05 PDT
Attachment 341044 [details] did not pass style-queue:


ERROR: Source/WebCore/page/mac/PageMac.mm:81:  One line control clauses should not use braces.  [whitespace/braces] [4]
Total errors found: 1 in 10 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 10 WebKit Commit Bot 2018-05-22 19:36:10 PDT
Comment on attachment 341044 [details]
Patch

Clearing flags on attachment: 341044

Committed r232098: <https://trac.webkit.org/changeset/232098>
Comment 11 Chris Dumez 2018-06-07 10:31:55 PDT
Comment on attachment 341044 [details]
Patch

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

> Source/WebCore/loader/cache/MemoryCache.cpp:336
> +            WTFLogAlways("pruneLiveResourcesToSize: trying to prune %p", current);

Did you mean to keep this in? I see this in my terminal all the time now.
Comment 12 Simon Fraser (smfr) 2018-06-07 11:51:22 PDT
I did not, sorry. I'll remove it.
Comment 13 Simon Fraser (smfr) 2018-06-07 19:02:34 PDT
> > Source/WebCore/loader/cache/MemoryCache.cpp:336
> > +            WTFLogAlways("pruneLiveResourcesToSize: trying to prune %p", current);
> 
> Did you mean to keep this in? I see this in my terminal all the time now.

Removed in r232597
Comment 14 Radar WebKit Bug Importer 2018-06-07 19:03:25 PDT
<rdar://problem/40920513>