<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Web Inspector: Option+Click not jumping to resource"
href="https://bugs.webkit.org/show_bug.cgi?id=146498#c4">Comment # 4</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Web Inspector: Option+Click not jumping to resource"
href="https://bugs.webkit.org/show_bug.cgi?id=146498">bug 146498</a>
from <span class="vcard"><a class="email" href="mailto:joepeck@webkit.org" title="Joseph Pecoraro <joepeck@webkit.org>"> <span class="fn">Joseph Pecoraro</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=256271&action=diff" name="attach_256271" title="Patch">attachment 256271</a> <a href="attachment.cgi?id=256271&action=edit" title="Patch">[details]</a></span>
Patch
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=256271&action=review">https://bugs.webkit.org/attachment.cgi?id=256271&action=review</a>
<span class="quote">> Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorTokenTrackingController.js:355
> + if (this._delegate && typeof this._delegate.tokenTrackingControllerHighlightedRangeWasClicked === "function")
> + this._delegate.tokenTrackingControllerHighlightedRangeWasClicked(this);</span >
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.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>