Bug 62860 – Web Inspector: Highlighting broken while navigating the tabs

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

[Andrey Adaikin]

1. Go to a web page, open the DevTools on the Console tab
2. Hit refresh (Ctrl+F5) on the console tab
3. Go to the scripts tab, and observe the script is not highlighted

NOTE: This bug exists on Chrome 14.0.795.0 (Developer Build 4a50d71),
but not in Chrome 13.0.782.15 (Official Build 88529) dev.

[Andrey Adaikin]

Updated steps to repro:

1. Go to a web page, open the DevTools on the Scripts tab
2. Open console by pressing the ESC button
3. Go to Console tab
4. Hit refresh (Ctrl+F5) on the Console tab
5. Go to the Scripts tab, and observe that the script is not highlighted

[Andrey Adaikin]

I remember I fixed the very same bug here:
http://trac.webkit.org/browser/trunk/Source/WebCore/inspector/front-end/inspector.js#L1195

but somebody added this code snippet before it:

if (!panel.canShowAnchorLocation(anchor))
  return false;

thus I guess the bug appeared again.

[Andrey Adaikin]

And there should be no horizontal scroll in the Scripts panel bar to repro this bug.

[Pavel Podivilov]

(In reply to comment #2)
> I remember I fixed the very same bug here:
> http://trac.webkit.org/browser/trunk/Source/WebCore/inspector/front-end/inspector.js#L1195
> 
> but somebody added this code snippet before it:
> 
> if (!panel.canShowAnchorLocation(anchor))
>   return false;
> 
> thus I guess the bug appeared again.

It's unrelated since you don't click on the link in your scenario, you just switch tabs. I think this bug was there for a very long time.

[Andrey Adaikin]

Yes, you right. Then the this.drawer.immediatelyFinishAnimation() call should be removed from there, and added to the place where you switch the panels.

[Pavel Podivilov]

Created attachment 98689 [details]
Patch.

[Pavel Podivilov]

Committed r90096: <http://trac.webkit.org/changeset/90096>