Bug 1307127 - Pass along pageTimeout when polling for ready state from chrome; r?automatedtester
The page loading timeout is not included in the message sent to
Marionette:pollForReadyState from chrome after a remoteness change occurs.
The call is appended to the pending commands and re-issued/flushed when
Marionette:listenersAttached fires after a remoteness change.
MozReview-Commit-ID: 4UMD4x9ugMA
--- a/testing/marionette/driver.js
+++ b/testing/marionette/driver.js
@@ -969,16 +969,17 @@ GeckoDriver.prototype.get = function*(cm
// TODO(ato): Bug 1242595
let id = this.listener.activeMessageId;
// If a remoteness update interrupts our page load, this will never return
// We need to re-issue this request to correctly poll for readyState and
// send errors.
this.curBrowser.pendingCommands.push(() => {
cmd.parameters.command_id = id;
+ cmd.parameters.pageTimeout = this.pageTimeout;
this.mm.broadcastAsyncMessage(
"Marionette:pollForReadyState" + this.curBrowser.curFrameId,
cmd.parameters);
});
yield get;
break;
