Closed Bug 1093048 Opened 10 years ago Closed 10 years ago

add string overload for nsIDOMWindowUtils::removeSheet method (e10s)

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal
Points:
1

Tracking

()

RESOLVED FIXED
mozilla36
Iteration:
36.2
Tracking Status
e10s + ---

People

(Reporter: zombie, Assigned: mossop)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1089230 +++

i forgot to ask for removeSheet() method in bug 1089230.
Attached patch patchSplinter Review
Assignee: nobody → dtownsend+bugmail
Status: NEW → ASSIGNED
Attachment #8516229 - Flags: review?(bobbyholley)
tracking-e10s: --- → +
Component: General → DOM
Product: Add-on SDK → Core
Flags: qe-verify-
Flags: firefox-backlog+
Comment on attachment 8516229 [details] [diff] [review]
patch

Review of attachment 8516229 [details] [diff] [review]:
-----------------------------------------------------------------

Hrm. 'RemoveSheetFromURIString' has an unfortunate semantic connotation that the sheet is being removed from the string (whereas 'LoadSheetFromURIString' has no such ambiguity).

Can we change both of them to LoadSheetUsingURIString and RemoveSheetUsingURIString? r=me with that.
Attachment #8516229 - Flags: review?(bobbyholley) → review+
https://hg.mozilla.org/mozilla-central/rev/fb567390cb92
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: