Allow switching to Browser elements when in chrome

RESOLVED FIXED in mozilla37

Status

Testing
Marionette
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: automatedtester, Assigned: automatedtester)

Tracking

({ateam-marionette-server})

unspecified
mozilla37
x86
Mac OS X
ateam-marionette-server
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [affects=loop])

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

3 years ago
Currently Marionette only looks for XUL:Browser elements when switching but but we also have Browser elements as frames. This means that loop can't switch to their hello frame
(Assignee)

Updated

3 years ago
Blocks: 1095303
(Assignee)

Comment 2

3 years ago
Created attachment 8532539 [details]
MozReview Request: bz://1107915/AutomatedTester
Attachment #8532539 - Flags: review?(mdas)
(Assignee)

Comment 3

3 years ago
/r/1169 - Bug 1107915: Allow switching to Browser elements from Marionette; r=mdas
/r/1171 - remove items that shouldn't have been up for review

Pull down these commits:

hg pull review -r 6980775b909e30999cec0667b01e7243df2e733d
(Assignee)

Comment 4

3 years ago
https://reviewboard.mozilla.org/r/1163/#review675

The second commit should cover some potential nits so check there too :)
Comment on attachment 8532539 [details]
MozReview Request: bz://1107915/AutomatedTester

r+ if it doesn't break Mn/Mnw
Attachment #8532539 - Flags: review?(mdas) → review+
(Assignee)

Updated

3 years ago
Assignee: nobody → dburns
https://hg.mozilla.org/mozilla-central/rev/16f8b6b0acff
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Thanks so much, guys; this fix is huge!
(Assignee)

Comment 11

3 years ago
Comment on attachment 8532539 [details]
MozReview Request: bz://1107915/AutomatedTester
Attachment #8532539 - Attachment is obsolete: true
Attachment #8618814 - Flags: review+
Attachment #8618815 - Flags: review+
(Assignee)

Comment 12

3 years ago
Created attachment 8618814 [details]
MozReview Request: Bug 1107915: Allow switching to Browser elements from Marionette; r=mdas
(Assignee)

Comment 13

3 years ago
Created attachment 8618815 [details]
MozReview Request: remove items that shouldn't have been up for review
You need to log in before you can comment on or make changes to this bug.