Bug 211244 - Web Inspector: Worker: should use the name of the worker if it exists
Summary: Web Inspector: Worker: should use the name of the worker if it exists
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Devin Rousso
URL:
Keywords: InRadar
: 164678 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-04-30 11:58 PDT by Devin Rousso
Modified: 2020-05-04 12:52 PDT (History)
11 users (show)

See Also:


Attachments
Patch (29.03 KB, patch)
2020-04-30 13:09 PDT, Devin Rousso
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Devin Rousso 2020-04-30 11:58:34 PDT
test.html
```
let worker = new Worker("worker.js", {name: "My Worker"});
```

worker.js
```
console.log(self.name);
```
Comment 1 Devin Rousso 2020-04-30 13:09:59 PDT
Created attachment 398084 [details]
Patch
Comment 2 EWS Watchlist 2020-04-30 13:10:42 PDT
This patch modifies the inspector protocol. Please ensure that any frontend changes appropriately use feature checks for new protocol features.
Comment 3 BJ Burg 2020-05-01 15:59:55 PDT
Comment on attachment 398084 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=398084&action=review

r=me great cleanup!

Please investigate Windows EWS failure:

C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\ForwardingHeaders\JavaScriptCore/InspectorFrontendDispatchers.h(351,10): message : see declaration of 'Inspector::WorkerFrontendDispatcher::workerCreated' (compiling source file C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-sources\UnifiedSource-84c9f43f-6.cpp) [C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]

> LayoutTests/inspector/worker/runtime-basic-expected.txt:13
>  Target - Page - location and href - Location: inspector/worker/runtime-basic.html
>  

Very nice!
Comment 4 Devin Rousso 2020-05-04 11:16:49 PDT
(In reply to Brian Burg from comment #3)
> Please investigate Windows EWS failure:
> 
> C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\ForwardingHeaders\JavaScriptCore/InspectorFrontendDispatchers.h(351,10): message : see declaration of 'Inspector::WorkerFrontendDispatcher::workerCreated' (compiling source file C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-sources\UnifiedSource-84c9f43f-6.cpp) [C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]

The windows bot doesn't seem to update/regenerate DerivedSources properly.  This has happened a bunch of times when I've made changes to the protocol.  A clean build has always fixed the issue.
Comment 5 Devin Rousso 2020-05-04 11:39:00 PDT
*** Bug 164678 has been marked as a duplicate of this bug. ***
Comment 6 EWS 2020-05-04 12:51:33 PDT
Committed r261104: <https://trac.webkit.org/changeset/261104>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 398084 [details].
Comment 7 Radar WebKit Bug Importer 2020-05-04 12:52:21 PDT
<rdar://problem/62857411>