Bug 34335

Summary:

ScriptCallFrame's do not get line numbers/source id's if the caller is unkown.

Product:

WebKit

Reporter:

Ben Murdoch <benm>

Component:

JavaScriptCore

Assignee:

Ben Murdoch <benm>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

dglazkov, ggaren, sam

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Proposed patch and updated layout tests.	darin: review+, benm: commit-queue-

Ben Murdoch

Reported 2010-01-29 10:06:33 PST

When we append a StackCallFrame to the ScriptCallStack, if the calling function is unknown we do not supply the CallFrame with any source id or line number information although this data is available. Instead, an empty source id and line number of 0 is supplied. I propose a patch to add the correct line number and source ID to the callframe. Patch and updates to layout tests to follow.

Attachments
Proposed patch and updated layout tests. (6.50 KB, patch) 2010-02-01 04:44 PST, Ben Murdoch	darin: review+ benm: commit-queue-	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Ben Murdoch

Comment 1 2010-02-01 04:44:16 PST

Created attachment 47829 [details] Proposed patch and updated layout tests.

Darin Adler

Comment 2 2010-02-01 10:00:54 PST

Comment on attachment 47829 [details] Proposed patch and updated layout tests. > + * platform/gtk/inspector/console-log-before-inspector-open-expected.txt: Removed. > + * platform/mac/inspector/console-log-before-inspector-open-expected.txt: Removed. > + * platform/win/inspector/console-log-before-inspector-open-expected.txt: Removed. Change log doesn't say why you did this. I presume that these platform-specific results were actually just working around the bug?

Ben Murdoch

Comment 3 2010-02-01 10:03:56 PST

Thanks for the review! (In reply to comment #2) > (From update of attachment 47829 [details]) > > + * platform/gtk/inspector/console-log-before-inspector-open-expected.txt: Removed. > > + * platform/mac/inspector/console-log-before-inspector-open-expected.txt: Removed. > > + * platform/win/inspector/console-log-before-inspector-open-expected.txt: Removed. > > Change log doesn't say why you did this. I presume that these platform-specific > results were actually just working around the bug? Precisely - I can update the ChangeLog and land manually, if you like? Thanks, Ben

Ben Murdoch

Comment 4 2010-02-01 10:29:57 PST

Comment on attachment 47829 [details] Proposed patch and updated layout tests. Will land manually.

Ben Murdoch

Comment 5 2010-02-02 02:15:21 PST

Landed as r54138. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.