Bug 171569 – REGRESSION (r215630): Web Inspector: Option-Click on URL in Styles sidebar does not work

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

[Nikita Vasilyev]

-------
Inspected URL:        https://webkit.org/
Loading completed:    true
Frontend User Agent:  Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_4) AppleWebKit/604.1.21+ (KHTML, like Gecko)

Uncaught Exceptions:
 - TypeError: frame.resourceForURL is not a function. (In 'frame.resourceForURL(simplifiedURL, searchChildFrames)', 'frame.resourceForURL' is undefined) (at Main.js:856:91)
    openURL @ Main.js:856:91
    tokenTrackingControllerHighlightedRangeWasClicked @ CSSStyleDeclarationTextEditor.js:1717:33
    _mouseButtonWasReleasedOverEditor @ CodeMirrorTokenTrackingController.js:425:89
    handleEvent @ CodeMirrorTokenTrackingController.js:253:51
-------

* STEPS TO REPRODUCE
1. Open https://webkit.org/.
2. Inspect the WebKit logo in top left corner (h1.logo).
3. In the Styles sidebar, find "background-image: url('images/webkit.svg');" and option-click on the image URL.

* EXPECTED
Image opens in the Resources tab.

[Radar WebKit Bug Importer]

<rdar://problem/31946367>

[Devin Rousso]

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;
        }

[Nikita Vasilyev]

I confirm it regressed in https://trac.webkit.org/changeset/215630/webkit.

[Nikita Vasilyev]

(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?

[Devin Rousso]

Created attachment 308994 [details]
Patch

[Joseph Pecoraro]

Comment on attachment 308994 [details]
Patch

r=me

[WebKit Commit Bot]

Comment on attachment 308994 [details]
Patch

Clearing flags on attachment: 308994

Committed r216166: <http://trac.webkit.org/changeset/216166>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.