* SUMMARY Option+Click not jumping to resource. * STEPS TO REPRODUCE 1. Inspect <body> on <http://bogojoker.com> 2. In the Styles sidebar, Option+Click anything in the CSS (or the pattern.png resource) => no jump to resource
<rdar://problem/21627454>
Created attachment 256271 [details] Patch
Comment on attachment 256271 [details] Patch The force tab stuff was added because I was seeing tab switches during cookie restore. Maybe we don't need it now because cookie restore is only done when the tab is selected?
Comment on attachment 256271 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=256271&action=review > Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorTokenTrackingController.js:355 > + if (this._delegate && typeof this._delegate.tokenTrackingControllerHighlightedRangeWasClicked === "function") > + this._delegate.tokenTrackingControllerHighlightedRangeWasClicked(this); This is not commented in the ChangeLog. We couldn't determine why the setTimeout was needed... but it looks like it was there to deal with CodeMirror. In any case, if you didn't see a regression with this change it seems find to make.
Comment on attachment 256271 [details] Patch Doing some quick searches I found a few call sites that were missed. (Maybe you just searched WebInspector.foo and missed these which don't 100% match that call site signature): Base/Main.js 1535: this.showMainFrameDOMTree(event.data.node, true); Base/Main.js 1138: this.showSourceCodeForFrame(frame.id, true); Thanks for looking into this!
(In reply to comment #3) > Comment on attachment 256271 [details] > Patch > > The force tab stuff was added because I was seeing tab switches during > cookie restore. Maybe we don't need it now because cookie restore is only > done when the tab is selected? That was exactly what I was thinking and why I suggested this approach to Devin. I cannot think of any case where we don't want to switch to a relevant tab when trying to show a new content view. The idea with window.event was clever, but I can't think of cases where we would trigger switching content views that wasn't somehow user gesture triggered. Even if we end up needing to keep/restore this parameter, I'd rather see it negated, and have the default behavior allow for switching tabs.
Created attachment 256290 [details] Patch
Comment on attachment 256290 [details] Patch Clearing flags on attachment: 256290 Committed r186466: <http://trac.webkit.org/changeset/186466>
All reviewed patches have been landed. Closing bug.