[e10s] Marionette.navigate() hangs when first tab gets closed with: NS_ERROR_ILLEGAL_VALUE in nsIMessageSender.sendAsyncMessage

RESOLVED FIXED in Firefox 41

Status

defect
--
critical
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: whimboo, Unassigned)

Tracking

({hang, regression})

41 Branch
mozilla41
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox40 unaffected, firefox41 fixed)

Details

Attachments

(1 attachment)

marionette.navigate() will hang in getting the current URL after the first tab has been closed in e10s mode. It does not happen in non-e10s mode. To reproduce this issue simply run the attached testcase with Marionette and e10s enabled.

Here the Javascript failure:

JavaScript error: chrome://marionette/content/listener.js, line 374: NS_ERROR_ILLEGAL_VALUE: Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsIMessageSender.sendAsyncMessage]

With Marionette logging turned on you can see it fails when trying to execute "get":

*** Loading URL which will freeze the execution of the test
1434962441252	Marionette	DEBUG	conn4 -> ({"to":"0","sessionId":"{d5939d11-da6f-4471-8373-fe04c4bfcf62}","name":"getContext"})
1434962441253	Marionette	DEBUG	conn4 client <- ({"from":"0","value":"chrome","status":"success","sessionId":"{d5939d11-da6f-4471-8373-fe04c4bfcf62}"})
1434962441254	Marionette	DEBUG	conn4 -> ({"to":"0","sessionId":"{d5939d11-da6f-4471-8373-fe04c4bfcf62}","name":"setContext","parameters":{"value":"content"}})
1434962441254	Marionette	DEBUG	conn4 client <- ({"from":"0","ok":true})
1434962441255	Marionette	DEBUG	conn4 -> ({"to":"0","sessionId":"{d5939d11-da6f-4471-8373-fe04c4bfcf62}","name":"get","parameters":{"url":"http://mozqa.com"}})

After the last line Marionette hangs and does not proceed with the test anymore. A quick test has shown that this is a regression for Firefox 41. I will check for a closer range.
Summary: [e10s] Marionette.navigate() hangs when first tab gets closed with: → [e10s] Marionette.navigate() hangs when first tab gets closed with: NS_ERROR_ILLEGAL_VALUE in nsIMessageSender.sendAsyncMessage
This regressed between 06-11 and 06-12. The pushlog from mozilla-central:

https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=9ebd530c5843&tochange=8cf9d3e497f9

There are no e10s related changes in that inbound merge which could have caused such changed behavior. So I strongly feel that the fix from Chris on bug 1169600 regressed the behavior in Marionette itself.

Chris, can you please have a look at this? Thanks.
Blocks: 1169603
This is the underlying problem in bug 1169600 (which was not addressed in the original patch), I'm going to post a new fix there.
Issue got fixed in bug 1169600.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.