Closed
Bug 1136378
Opened 10 years ago
Closed 10 years ago
Avoid using a CPOW to identify marionette's content listener
Categories
(Remote Protocol :: Marionette, defect)
Tracking
(firefox39 fixed)
RESOLVED
FIXED
mozilla39
Tracking | Status | |
---|---|---|
firefox39 | --- | fixed |
People
(Reporter: chmanchester, Assigned: chmanchester)
References
Details
Attachments
(1 file, 1 obsolete file)
This is a follow up for bug 1125377.
Assignee | ||
Comment 1•10 years ago
|
||
Review board felt strongly this needed a new bug number to work on.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=1b0f90421f48
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8bd8db137b12
Assignee | ||
Comment 2•10 years ago
|
||
/r/4261 - Bug 1136378 - Use a browser's outerWindowId property rather than a CPOW to identify marionette's content listeners.
Pull down this commit:
hg pull review -r 403e38341d1925a5fb8ea976f5213988089afd02
Attachment #8568786 -
Flags: review?(mconley)
Assignee | ||
Comment 3•10 years ago
|
||
:mconley -- looking at the implementation of outerWindowID landed in bug 1077168 this is what I need here, but as this is recently landed I wanted your sign off on whether this is something that's reasonable use and reasonably unlikely to bounce. Thanks!
Comment 4•10 years ago
|
||
Comment on attachment 8568786 [details]
MozReview Request: bz://1136378/chmanchester
https://reviewboard.mozilla.org/r/4259/#review3425
Yeah! Good catch! :D
Attachment #8568786 -
Flags: review?(mconley) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Try was taking a while, and inbound was open, so:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ca4f427de026
Comment 6•10 years ago
|
||
Assignee: nobody → cmanchester
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Assignee | ||
Comment 7•10 years ago
|
||
Attachment #8568786 -
Attachment is obsolete: true
Attachment #8619583 -
Flags: review+
Assignee | ||
Comment 8•10 years ago
|
||
Updated•2 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•