Add diagnostic logging to track the number of page loads.
<rdar://problem/18173017>
<rdar://problem/18820041>
Created attachment 240649 [details] Proposed patch.
Attachment 240649 [details] did not pass style-queue: ERROR: Source/WebCore/loader/FrameLoader.cpp:47: Alphabetical sorting problem. [build/include_order] [4] Total errors found: 1 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 240650 [details] Updated patch > ERROR: Source/WebCore/loader/FrameLoader.cpp:47: Alphabetical sorting problem. [build/include_order] [4] Oh the horror!
Comment on attachment 240650 [details] Updated patch Clearing flags on attachment: 240650 Committed r175375: <http://trac.webkit.org/changeset/175375>
All reviewed patches have been landed. Closing bug.
Comment on attachment 240650 [details] Updated patch View in context: https://bugs.webkit.org/attachment.cgi?id=240650&action=review > Source/WebCore/loader/FrameLoader.cpp:2270 > + page->chrome().client().logDiagnosticMessage(DiagnosticLoggingKeys::pageLoadedKey(), emptyString(), error.isNull() ? DiagnosticLoggingKeys::passKey() : DiagnosticLoggingKeys::failKey()); This seems to log for each frame load (included subframes). Was this the intention?