Closed Bug 1392339 Opened 7 years ago Closed 7 years ago

API docs should not use nsIDOMWindow

Categories

(Remote Protocol :: Marionette, enhancement)

Version 3
enhancement
Not set
normal

Tracking

(firefox57 fixed)

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: ato, Assigned: ato)

References

Details

Attachments

(2 files)

The nsIDOMWindow is an XPCOM type and not part of the web platform
we’re using in content scripts.
This doesn’t block marionette-window-tracking in reality, but I
am attempting to split that WIP patch up in landable chunks, so
slightly overloading the meaning of a blocking bug in this regard.
Assignee: nobody → ato
Status: NEW → ASSIGNED
Depends on: 1392281
Comment on attachment 8899550 [details]
Bug 1392339 - Fix misuse of nsIDOMWindow in API docs.

https://reviewboard.mozilla.org/r/170844/#review176346
Attachment #8899550 - Flags: review?(dburns) → review+
Comment on attachment 8899551 [details]
Bug 1392339 - Fix misuse of nsIDOMElement in API docs.

https://reviewboard.mozilla.org/r/170846/#review176348
Attachment #8899551 - Flags: review?(dburns) → review+
Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/771cd906e137
Fix misuse of nsIDOMWindow in API docs. r=automatedtester
https://hg.mozilla.org/integration/autoland/rev/5b82c36142fb
Fix misuse of nsIDOMElement in API docs. r=automatedtester
https://hg.mozilla.org/mozilla-central/rev/771cd906e137
https://hg.mozilla.org/mozilla-central/rev/5b82c36142fb
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: