Bug 170890

Summary: Web Inspector: XHR breakpoints restored from settings do not appear in the sidebar
Product: WebKit Reporter: Matt Baker <mattbaker>
Component: Web InspectorAssignee: Matt Baker <mattbaker>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, commit-queue, inspector-bugzilla-changes, joepeck, rniwa, webkit-bug-importer
Priority: P1 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews104 for mac-elcapitan-wk2 none

Description Matt Baker 2017-04-16 12:46:34 PDT 
Summary:
XHR breakpoints restored from settings do not appear in the sidebar. Breakpoints added from the UI appear as expected.

Steps to Reproduce:
1. Open Debugger tab
2. Add an XHR breakpoint
  => Breakpoint added to sidebar
3. Reload the page
  => Breakpoint disappears

Note that the breakpoint will still cause the page to break, since the problem is not in DOMDebuggerManager.
Comment 1 Radar WebKit Bug Importer 2017-04-16 12:47:37 PDT 
<rdar://problem/31649271>
Comment 2 Matt Baker 2017-04-16 12:50:32 PDT 
Created attachment 307236 [details]
Patch
Comment 3 Build Bot 2017-04-16 14:08:56 PDT 
Comment on attachment 307236 [details]
Patch

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

New failing tests:
webrtc/multi-video.html
Comment 4 Build Bot 2017-04-16 14:08:57 PDT 
Created attachment 307239 [details]
Archive of layout-test-results from ews104 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 5 Joseph Pecoraro 2017-04-17 12:02:38 PDT 
Comment on attachment 307236 [details]
Patch

r=me
Comment 6 WebKit Commit Bot 2017-04-17 12:30:59 PDT 
Comment on attachment 307236 [details]
Patch

Clearing flags on attachment: 307236

Committed r215422: <http://trac.webkit.org/changeset/215422>
Comment 7 WebKit Commit Bot 2017-04-17 12:31:00 PDT 
All reviewed patches have been landed.  Closing bug.