Bug 178470

Summary: Check notifyPagesWhenTelemetryWasCaptured before notifying pages if there is no WebPageProxy
Product: WebKit Reporter: John Wilander <wilander>
Component: WebKit2Assignee: John Wilander <wilander>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, commit-queue, webkit-bug-importer, wilander
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

John Wilander
Reported 2017-10-18 10:00:58 PDT
There's a missing check of the test setting notifyPagesWhenTelemetryWasCaptured when there's no WebPageProxy and we notify pages of a zero result.
Attachments
Patch (1.42 KB, patch)
2017-10-18 10:04 PDT, John Wilander 		no flags
DetailsFormatted DiffDiff
Patch (1.44 KB, patch)
2017-10-18 10:08 PDT, John Wilander 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
John Wilander
Comment 1 2017-10-18 10:01:16 PDT
<rdar://problem/34417518>
John Wilander
Comment 2 2017-10-18 10:04:31 PDT
Created attachment 324126 [details] Patch
John Wilander
Comment 3 2017-10-18 10:08:22 PDT
Created attachment 324127 [details] Patch
Blaze Burg
Comment 4 2017-10-18 10:51:33 PDT
Comment on attachment 324127 [details] Patch r=me
John Wilander
Comment 5 2017-10-18 10:54:24 PDT
Comment on attachment 324127 [details] Patch Thanks, Brian!
WebKit Commit Bot
Comment 6 2017-10-18 11:21:19 PDT
Comment on attachment 324127 [details] Patch Clearing flags on attachment: 324127 Committed r223616: <https://trac.webkit.org/changeset/223616>
WebKit Commit Bot
Comment 7 2017-10-18 11:21:21 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.