Bug 39923 – Web Inspector: [REGRESSION] caller locations are not shown on Timeline Panel.

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

[Pavel Feldman]

A nasty regression that kills source locations for all records on timeline. I am reverting frontend changes made in http://trac.webkit.org/changeset/60083#file14 to fix it.

[Pavel Feldman]

Created attachment 57413 [details]
[PATCH] Proposed fix.

WebCore/inspector/front-end/TimelinePanel.js: 
 +              contentHelper._appendLinkRow(WebInspector.UIString("Caller"), this.callerScriptName, callSite.lineNumber);


This would seem to be the bug right here.

this.callerScriptName is null. I believe it should instead be callsite.scriptName.

This may have been an issue with hand merging this file. 

@Pavel: I recall when you were landing it you said that timelinepanel.js had a conflict with another change that went in while https://bugs.webkit.org/show_bug.cgi?id=37502 was sitting in the commit queue. I believe that the merge had a small mistake.

Notice in https://bugs.webkit.org/attachment.cgi?id=56398&action=review the patch has the correct piece of code.

I recommend in stead of reverting, just try changing that one line and testing to see if the bug is resolved.

[WebKit Commit Bot]

Comment on attachment 57413 [details]
[PATCH] Proposed fix.

Clearing flags on attachment: 57413

Committed r60412: <http://trac.webkit.org/changeset/60412>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Pavel Feldman]

> Notice in https://bugs.webkit.org/attachment.cgi?id=56398&action=review the patch has the correct piece of code.
> 
> I recommend in stead of reverting, just try changing that one line and testing to see if the bug is resolved.

Yes. Sorry for this to sound like Jaime regressed it. Now it is clear to me that it was a poor merge on my side. Having said that, Jaime did have a small bug that was dropping callers for rendering events. Not from all of them though :P.

I reverted the change since I guess simply getting callserScriptName and line in the first line of formatting, was guaranteeing that old code was still functioning. So it is the shortest distance to the working code after the stack refactoring. The code still works and it works right, delta from the pre-stack state is just 4 lines!