Closed
Bug 1136378
Opened 9 years ago
Closed 9 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•9 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•9 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•9 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•9 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•9 years ago
|
||
Try was taking a while, and inbound was open, so: https://hg.mozilla.org/integration/mozilla-inbound/rev/ca4f427de026
Comment 6•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/ca4f427de026
Assignee: nobody → cmanchester
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Assignee | ||
Comment 7•9 years ago
|
||
Attachment #8568786 -
Attachment is obsolete: true
Attachment #8619583 -
Flags: review+
Assignee | ||
Comment 8•9 years ago
|
||
Updated•1 year ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•