1506513 - [remote-dbg-next] Could not inspect any worker

Status: RESOLVED FIXED

(DevTools :: about:debugging, enhancement, P1)

Description

[Daisuke Akatsuka (:daisuke)]

An error occur from JavaScript as below when click the inspect button for any workers.

```
JavaScript error: resource://devtools/client/framework/devtools-browser.js, line 384: TypeError: client.attachWorker is not a function
```

STRs:
1. Open about:debugging
2. Click an inspect button of a worker

ER:
Can inspect.

AR:
An error occurred.

This seems caused by bug 1497917.
We introduced client-wrapper from the bug, though gDevToolsBrowser.openWorkerToolbox expect debugger-client, we are sending client-wrapper. 
https://searchfox.org/mozilla-central/source/devtools/client/aboutdebugging-new/src/actions/debug-targets.js#65

Comment 1

[Julian Descottes [:jdescottes]]

filter on remote-debugging-next-move-m3-to-m2

Comment 2

[Julian Descottes [:jdescottes]]

filter on remote-debugging-next-move-m3-to-m2

Comment 3

[Julian Descottes [:jdescottes]]

filter on remote-debugging-next-move-m3-to-m2

Comment 4

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1506513: Use clientWrapper instead of client. r=jdescottes!

Comment 5

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1506513: Add worker mock test. r=jdescottes!

Depends on D13970

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/62dd0d1a37b7
Use clientWrapper instead of client. r=jdescottes
https://hg.mozilla.org/integration/autoland/rev/7b9332b0c24d
Add worker mock test. r=jdescottes

Comment 7

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/62dd0d1a37b7
https://hg.mozilla.org/mozilla-central/rev/7b9332b0c24d