WebInspector: Adds InspectorTimelineAgent instrumentation for <script> tag processing

[Eric Ayers]

This change adds another instrumentation point for the InspectorTimelineAgent - reporting the time spent processing a script tag.

In order to report the URL and line number for the tag, I updated SourceSourceCode to provide accessors for these values.  I could have de-referenced the JSC object, but that would cause a fork with Chrome.

[Eric Ayers]

Created attachment 42031 [details]
WebInspector: Adds instrumentation of the script tag, and getters to ScriptSourceCode

[Timothy Hatcher]

Comment on attachment 42031 [details]
WebInspector: Adds instrumentation of the script tag, and getters to ScriptSourceCode


> +        timelineAgent->willScriptTag(sourceCode.url().isNull() ? String() : sourceCode.url().string(), sourceCode.startLine());
> +        timelineAgent->didScriptTag();
> +        ScriptTagTimelineRecordType = 10,

The wording is weird here. Would be better as willEvaluateScriptTag/didEvaluateScriptTag/EvaluateScriptTagTimelineRecordType.

Otherwise r+.

[Eric Ayers]

Created attachment 42042 [details]
WebInspector: Renames ScriptTagTimelineRecord to EvaluateScriptTagTimelineRecord

[WebKit Commit Bot]

Comment on attachment 42042 [details]
WebInspector: Renames ScriptTagTimelineRecord to EvaluateScriptTagTimelineRecord

Clearing flags on attachment: 42042

Committed r50230: <http://trac.webkit.org/changeset/50230>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.