Bug 226664 – Web Inspector: Uncaught Exception: undefined is not an object (evaluating 'InspectorBackend.Enum.Page.ResourceType')

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Devin Rousso]

Uncaught Exception in Web Inspector.

Steps to Reproduce:
1. inspect the twitter.com service worker
2. reload the page

Uncaught Exceptions:
-----------------------
 - undefined is not an object (evaluating 'InspectorBackend.Enum.Page.ResourceType') (at NetworkManager.js:1064:68)
    _addNewResourceToFrameOrTarget @ NetworkManager.js:1064:68
    resourceRequestWillBeSent @ NetworkManager.js:664:55
    requestWillBeSent @ NetworkObserver.js:48:52
    _dispatchEvent @ Connection.js:210:26
    dispatch @ Connection.js:79:32
    dispatch @ InspectorBackend.js:232:52
    ? @ MessageDispatcher.js:42:34

Additional Details:
    cause --> An uncaught exception was thrown while handling event: Network.requestWillBeSent
-----------------------

Notes:
Inspected URL:        blob:https://twitter.com/27274f10-3876-4d84-b731-eedfd3b3c46e
Loading completed:    true
Frontend User Agent:  Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko)
Dispatch Source:      Protocol Event

Protocol Event:
{"method":"Network.requestWillBeSent","params":{"requestId":"0.145","frameId":"","loaderId":"","documentURL":"https://twitter.com/i/api/1.1/branch/init.json","request":{"url":"https://twitter.com/i/api/1.1/branch/init.json","method":"POST","headers":{"Accept":"*/*","Content-Type":"application/json","Authorization":"Bearer AAAAAAAAAAAAAAAAAAAAANRILgAAAAAAnNwIzUejRCOuH5E6I8xnZz4puTs%3D1Zv7ttfk8LF81IUq16cHjhLTvJu4FA33AGWWjCpTnA","x-csrf-token":"bd6f7ed739f0e02b034fe8c618f37c6f","x-twitter-active-us…

[Devin Rousso]

Created attachment 430619 [details]
Patch

[Joseph Pecoraro]

Comment on attachment 430619 [details]
Patch

Testable?

[Devin Rousso]

(In reply to Joseph Pecoraro from comment #2)
> Testable?
Not really, as the tests are always either `"page"` or `"web-page"` (or don't use `InspectorBackend` at all) and therefore already had all of the domains that had enums.  We maybe could do something like checking that a WK1 test has `InspectorBackend.Enum.Target.TargetInfoType` (since that's only enabled for `"web-page"`, but that seems kinda awful :/

[Devin Rousso]

(In reply to Devin Rousso from comment #3)
> (In reply to Joseph Pecoraro from comment #2)
> > Testable?
> 
> Not really, as the tests are always either `"page"` or `"web-page"` (or don't use `InspectorBackend` at all) and therefore already had all of the domains that had enums.  We maybe could do something like checking that a WK1 test has `InspectorBackend.Enum.Target.TargetInfoType` (since that's only enabled for `"web-page"`, but that seems kinda awful :/

Oh wait no I'm completely crazy yes we can add a test.  Just check for `InspectorBackend.Enum.ServiceWorker` (even though there are no enums in `ServiceWorker`) :)

[Devin Rousso]

Created attachment 430627 [details]
Patch

[EWS]

Committed r278512 (238511@main): <https://commits.webkit.org/238511@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 430627 [details].

[Radar WebKit Bug Importer]

<rdar://problem/78892937>