Bug 91534

Summary:	Web Inspector: inspectorinstrumentation::diddestroyworker is never called
Product:	WebKit	Reporter:	Hanna <hanma>
Component:	Web Inspector (Deprecated)	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED INVALID
Severity:	Normal	CC:	apavlov, bweinstein, joepeck, keishi, kpiascik, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority:	P2
Version:	528+ (Nightly build)
Hardware:	All
OS:	All
Bug Depends on:
Bug Blocks:	89692

Hanna

Reported 2012-07-17 12:43:32 PDT

inspectorinstrumentation::diddestroyworker is never called

Attachments
Add attachment proposed patch, testcase, etc.

Hanna

Comment 1 2012-07-18 12:59:11 PDT

So I looked into it a little. The current inspectorinstrumentation::diddestroyworker is called in ~abstrackworker, which sharedworker inherits, though it wasn't even called on sharedworker's destruction What I am thinking is to move the instrumentation of diddestroyworker to injectedscripthost, which is called via injectedfakeworker.js,and the instrumentation should be called on fakeworker destruction. Please let me know if this is be valid as I am not familiar with fakeworker, thank you

Hanna

Comment 2 2012-07-19 11:33:44 PDT

apparently this is fixed in the latest chromium build, can someone let me know which patch fixes this? thanks a bunch

Hanna

Comment 3 2012-07-27 09:53:28 PDT

this instrumentation is deleted

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