1106702 - enable devtools/scratchpad tests with e10s

Status: RESOLVED FIXED

(DevTools Graveyard :: Scratchpad, defect)

Description

[Heather Arthur [:harth]]

All the scratchpad tests are disabled in e10s right now.

Comment 1

[Panos Astithas (he/him) [:past] (please ni?)]

Attachment: Make Scratchpad's reloadAndRun use the RDP and enable scratchpad tests with e10s

Fixing this wasn't too hard after all, there was only reloadAndRun that had to be modified to use the remote debugging protocol instead of reaching into content directly.

Comment 2

[Heather Arthur [:harth]]

Comment on attachment 8536622 [details] [diff] [review]
Make Scratchpad's reloadAndRun use the RDP and enable scratchpad tests with e10s

Review of attachment 8536622 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good as long as try is good. An e10s try run would be good as well.

Comment 4

[Panos Astithas (he/him) [:past] (please ni?)]

Try and e10s try were green, apart from the expected leaks in debug mode:

https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=e89c9e50104c
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=e0506fd1f76e

Landed:

https://hg.mozilla.org/integration/fx-team/rev/c34f43a2a072

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c34f43a2a072