Bug 45756 – Web Inspector: Links from "Profiles" don't lead to "Scripts"

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Alexei Masterov]

Steps to reproduce:

1. navigate to a page with javascript (http://www.google.com/ for example)
2. open inspector by pressing CMD+ALT+i
3. select the "Profiles" tab, enable profiling if necessary, and click the Record button to record the CPU profile
4. execute some javascript action and click the Record button again to stop recording
5. Click on a link to the source code on the right hand side of the panel (see attached screenshot)

Desired outcome:
Inspector opens the "Scripts" panel with the function call selected

What happens instead:
"Resources" panel opens, and the corresponding file is selected

NOTE: if the scripts panel was accessed previously, the behavior may be different, so please use a freshly opened instance to repro

[Alexei Masterov]

Created attachment 67557 [details]
screenshot of where to click

[Yury Semikhatsky]

Created attachment 67573 [details]
Patch

[Pavel Feldman]

Comment on attachment 67573 [details]
Patch

Wrt WebInspector.documentMouseOver: I think it was used for lazy calculation of the tooltip text. Should be fine to nuke useless thingy.

[Joseph Pecoraro]

> (From update of attachment 67573 [details])
> Wrt WebInspector.documentMouseOver: I think it was used for lazy calculation
> of the tooltip text. Should be fine to nuke useless thingy.

It was originally added (with a real implementation) for:
<http://webkit.org/b/35574> Web Inspector: Nothing happens on navigation to resource via a link if resource tracking is disabled

How is that handled now? I don't know when the related code was removed, but it
does look like all the related code was removed.

[WebKit Commit Bot]

Comment on attachment 67573 [details]
Patch

Clearing flags on attachment: 67573

Committed r67525: <http://trac.webkit.org/changeset/67525>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.