[Webkit-unassigned] [Bug 89692] Web Inspector: show worker started and finished on timeline

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 12 08:48:18 PDT 2012


Pavel Feldman <pfeldman at chromium.org> changed:

           What    |Removed                     |Added
 Attachment #151710|review?                     |review-
               Flag|                            |

--- Comment #12 from Pavel Feldman <pfeldman at chromium.org>  2012-07-12 08:48:17 PST ---
(From update of attachment 151710)
View in context: https://bugs.webkit.org/attachment.cgi?id=151710&action=review

> Source/WebCore/inspector/InspectorInstrumentation.cpp:982
> +        timelineAgent->didCreateWorker(id, urlFromContext, isSharedWorker);

Passing two urls is confusing, we need to figure out why they differ.

> Source/WebCore/inspector/InspectorInstrumentation.cpp:1009
> +    if (InspectorTimelineAgent* timelineAgent = instrumentingAgents->inspectorTimelineAgent())

It would be nice if didDestroyWorker was called when it should be.

> Source/WebCore/inspector/front-end/TimelinePresentationModel.js:586
> +                contentHelper._appendTextRow(WebInspector.UIString("URL"), this.data["url"]);

I think Worker ID applies to the terminated info. Loading multiple workers from the same URL could be used for parallel data processing, so it happens more often than you might think.

> LayoutTests/inspector/timeline/timeline-web-workers.html:31
> +    InspectorTest.startTimeline(function() {

We always use named functions with { on the next line.

> LayoutTests/inspector/timeline/timeline-web-workers.html:37
> +        setTimeout(function(){InspectorTest.stopTimeline(step3);}, 5000);

Tests should not have timeouts. You should signal worker termination from the page to the front-end via logging something to the console and sniffing for the console message.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list