Closed Bug 811351 Opened 7 years ago Closed 7 years ago
We should call switch
_to _frame() on delete _session() for B2G marionette sessions
In B2G, if you switch into the browser frame and delete your session, then start a new session, you'll still be in the browser frame. I think it might be a good idea to switch back to the top-most frame (system app) when we delete the running session. Thoughts?
Yes, we should definitely do so either in deleteSession or newSession, in marionette-actors.js.
We should put the device back into a state that if a person starts it up again, and dont have defensive code to put them in a known state which they should anyway, then they can start from a good place. My gut is to switch it back to where it needs to be on deleteSession() but can be swayed either way.
This is a pretty easy fix; we just reset the current frame pointers to null in deleteSession, in both actors and listeners. This also fixes a typo in resetValues(), which resets the frame pointer in newSession in listeners. Will run on try before asking for review.
Comment on attachment 681198 [details] [diff] [review] Reset the current frame in deleteSession, try run was green
Update patch to focus default frames
Attachment #681550 - Flags: review?(mdas)
Attachment #681550 - Flags: review?(mdas) → review+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in before you can comment on or make changes to this bug.