Bug 185853 - Add notifyutil callbacks to dump the memory cache, and the list of live Pages and Documents
Summary: Add notifyutil callbacks to dump the memory cache, and the list of live Pages...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-05-21 17:18 PDT by Simon Fraser (smfr)
Modified: 2018-06-07 19:03 PDT (History)
11 users (show)

See Also:


Attachments
Patch (16.10 KB, patch)
2018-05-21 17:20 PDT, Simon Fraser (smfr)
keith_miller: review+
Details | Formatted Diff | Diff
Patch (16.29 KB, patch)
2018-05-22 14:38 PDT, Simon Fraser (smfr)
ews-watchlist: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews107 for mac-sierra-wk2 (2.96 MB, application/zip)
2018-05-22 16:11 PDT, EWS Watchlist
no flags Details
Patch (16.39 KB, patch)
2018-05-22 16:14 PDT, Simon Fraser (smfr)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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>