768258 - Set new_sandbox to False by default in marionette python client and server

Status: RESOLVED DUPLICATE of bug 1123506

(Remote Protocol :: Marionette, defect)

Description

[Malini Das [:mdas] - Away, not checking bugmail]

This will match Selenium's expectations

Comment 1

[Malini Das [:mdas] - Away, not checking bugmail]

I have a patch for this, but I'm waiting for Bug 761202's patch to land, since we'll have conflicts.

Comment 2

[Malini Das [:mdas] - Away, not checking bugmail]

Attachment: fix v0.1

This changes the default behaviour and updates tests accordingly

Comment 3

[Malini Das [:mdas] - Away, not checking bugmail]

Comment on attachment 638851 [details] [diff] [review]
fix v0.1

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

Argh, to mirror Selenium's behaviour, we'll need Marionette to know when a user changes the current window, even through execute_script. Right now, marionette doesn't listen for page load, and will use whatever window was last strictly refered to by the client (ie: via switch_to_frame, etc.). We'll need to have Marionette listening for page changes, or check that the window didn't change during a script call.