1478240 - Enable client/debugger/test/mochitest/browser_dbg_multiple-windows.js in e10s

Status: RESOLVED FIXED

(DevTools :: Debugger, enhancement, P3)

Description

[Julian Descottes [:jdescottes]]

Tests skipped in e10s are no longer running and should be enabled or removed.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1478240 - Convert browser_dbg_multiple-windows.js to use async/await instead of callbacks.

Review commit: https://reviewboard.mozilla.org/r/260724/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/260724/

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1478240 - Enable client/debugger/test/mochitest/browser_dbg_multiple-windows.js in e10s.

Review commit: https://reviewboard.mozilla.org/r/260726/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/260726/

Comment 3

[Alexandre Poirot [:ochameau]]

try:
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=a27cd45d711f5e38d70ebf1f5608d859630ca6ad

Comment 4

[Julian Descottes [:jdescottes]]

Comment on attachment 8996633 [details]
Bug 1478240 - Convert browser_dbg_multiple-windows.js to use async/await instead of callbacks.

https://reviewboard.mozilla.org/r/260724/#review267734

Great, thanks for the cleanup!

Comment 5

[Julian Descottes [:jdescottes]]

Comment on attachment 8996634 [details]
Bug 1478240 - Enable client/debugger/test/mochitest/browser_dbg_multiple-windows.js in e10s.

https://reviewboard.mozilla.org/r/260726/#review267742

Looks good and passes in e10s, thanks!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9847c25e9433
Convert browser_dbg_multiple-windows.js to use async/await instead of callbacks. r=jdescottes
https://hg.mozilla.org/integration/autoland/rev/0f9321db6320
Enable client/debugger/test/mochitest/browser_dbg_multiple-windows.js in e10s. r=jdescottes

Comment 7

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/9847c25e9433
https://hg.mozilla.org/mozilla-central/rev/0f9321db6320