Bug 150817 – Web Inspector: Search Results tab causes jump to Resources tab on reload

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Timothy Hatcher]

Steps:
1) Load any page
2) Do a search in the global search field to make a Search Results tab (a result needs found)
3) Go to the Console tab
4) Reload the page

The Resources tab will be switched to. It should not switch tabs and cookie restoring for the Search Results tab should not be done until it is active.

Views/ContentBrowserTabContentView.js:193:22: CONSOLE TRACE showRepresentedObject {"lineNumber":1248,"columnNumber":16}
0: showRepresentedObject(Views/ContentBrowserTabContentView.js:193:22)
1: showRepresentedObject(Base/Main.js:993:41)
2: showSourceCode(Base/Main.js:1033:31)
3: showOriginalOrFormattedSourceCodeTextRange(Base/Main.js:1054:24)
4: _treeElementSelected(Views/SearchSidebarPanel.js:384:68)
5: _treeElementSelected([native code])
6: select(Views/TreeOutline.js:1014:33)
7: revealAndSelect(Views/TreeOutline.js:1022:20)
8: (unknown)(Views/SearchSidebarPanel.js:154:61)
9: (unknown)([native code])
10: forEachMatch(Views/SearchSidebarPanel.js:120:25)
11: resourceCallback(Views/SearchSidebarPanel.js:149:33)
12: resourceCallback([native code])
13: _dispatchResponseToCallback(Protocol/InspectorBackend.js:233:27)
14: _dispatchResponse(Protocol/InspectorBackend.js:203:45)
15: dispatch(Protocol/InspectorBackend.js:87:35)
16: dispatchNextQueuedMessageFromBackend(Protocol/MessageDispatcher.js:42:34)

[Radar WebKit Bug Importer]

<rdar://problem/23362429>

[Timothy Hatcher]

It should also not jump to the Resources tab, it should use the ContentBrowser of the Search Results tab.

[Timothy Hatcher]

Created attachment 264898 [details]
Patch

[Blaze Burg]

Comment on attachment 264898 [details]
Patch

r=me

[WebKit Commit Bot]

Comment on attachment 264898 [details]
Patch

Clearing flags on attachment: 264898

Committed r192110: <http://trac.webkit.org/changeset/192110>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.