Summary: | REGRESSION (r215630): Web Inspector: Option-Click on URL in Styles sidebar does not work | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nikita Vasilyev <nvasilyev> | ||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, hi, inspector-bugzilla-changes, joepeck, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
URL: | https://webkit.org/ | ||||||
Attachments: |
|
Description
Nikita Vasilyev
2017-05-02 14:02:36 PDT
I think this regressed in r215630 <https://webkit.org/b/168709>. It looks like I forgot the `frame` parameter of `WebInspector.openURL` in CSSStyleDeclarationTextEditor.js:1717. It should be: // Special case option-clicking url(...) links. if (token && /\blink\b/.test(token.type)) { let url = token.string; let baseURL = sourceCodeLocation ? sourceCodeLocation.sourceCode.url : this._style.node.ownerDocument.documentURL; const frame = null; WebInspector.openURL(absoluteURL(url, baseURL), frame, options); return; } I confirm it regressed in https://trac.webkit.org/changeset/215630/webkit. (In reply to Devin Rousso from comment #2) > I think this regressed in r215630 <https://webkit.org/b/168709>. It looks > like I forgot the `frame` parameter of `WebInspector.openURL` in > CSSStyleDeclarationTextEditor.js:1717. It should be: > > // Special case option-clicking url(...) links. > if (token && /\blink\b/.test(token.type)) { > let url = token.string; > let baseURL = sourceCodeLocation ? > sourceCodeLocation.sourceCode.url : > this._style.node.ownerDocument.documentURL; > > const frame = null; > WebInspector.openURL(absoluteURL(url, baseURL), frame, options); > return; > } Yes, this solves the problem. Since you just wrote the fix, do you want to post a patch? Created attachment 308994 [details]
Patch
Comment on attachment 308994 [details]
Patch
r=me
Comment on attachment 308994 [details] Patch Clearing flags on attachment: 308994 Committed r216166: <http://trac.webkit.org/changeset/216166> All reviewed patches have been landed. Closing bug. |