Bug 164427 – Web Inspector: WorkerTarget's mainResource should be a Resource not a Script

• Status: NEW
• Component: Web Inspector

[Joseph Pecoraro]

Summary:
WorkerTarget's mainResource should be a Resource not a Script

This is the frontend portion once we address bug 164425.

Currently, all Worker resource loads are RawResourceRequests. Such resources do not always get saved. In the past that was fine because they are typically ephemeral (XHR,Fetch, etc). However Worker resources are more like <script src="..."> and we should expect Web Inspector to be informed of them. This impacts Web Inspector in a few different ways:

    - If Web Inspector is opened after a page has loaded
      - Worker Script Resources are not guaranteed to exist so the frontend may not know about them
      - Worker Script Resources may show up as a Script instead of a Resource
        - being Script and not Resource, don't get a Resource Details Sidebar
        - being Script and not Resource, the Network tab won't have Resource load data
    - Web Inspector has a workaround converting the type from XHR -> Script in some places

Notes:
- WorkerScriptLoader issues ThreadableLoader requests that it knows will be Scripts. It would be good to type such requests as Script early on.

[Radar WebKit Bug Importer]

<rdar://problem/29117462>

[Joseph Pecoraro]

I have workarounds for most of these issues that merely check if the WorkerTarget's mainResource (a Script) has an associated Resource. But it would be nice to eliminate all of these workarounds in the frontend.