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.
Created attachment 42031 [details] WebInspector: Adds instrumentation of the script tag, and getters to ScriptSourceCode
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+.
Created attachment 42042 [details] WebInspector: Renames ScriptTagTimelineRecord to EvaluateScriptTagTimelineRecord
Comment on attachment 42042 [details] WebInspector: Renames ScriptTagTimelineRecord to EvaluateScriptTagTimelineRecord Clearing flags on attachment: 42042 Committed r50230: <http://trac.webkit.org/changeset/50230>
All reviewed patches have been landed. Closing bug.