Closed Bug 1324904 Opened 9 years ago Closed 9 years ago

convert nsISupportsArray in mozilla/extensions/irc

Categories

(Other Applications Graveyard :: ChatZilla, defect)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aceman, Assigned: aceman)

References

Details

Attachments

(1 file)

mozilla/extensions/irc/xul/content/nsClipboard.js * nsISupportsArray getClipboardTransferable (Object aFlavourList) ; * returns a nsISupportsArray of the item on the clipboard const supportsIID = Components.interfaces.nsISupportsArray; mozilla/extensions/irc/xul/content/nsDragAndDrop.js * a reference to a function that returns a nsISupportsArray of nsITransferables
Attached patch patchSplinter Review
Attachment #8820412 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8820412 [details] [diff] [review] patch Review of attachment 8820412 [details] [diff] [review]: ----------------------------------------------------------------- r=me, thanks!
Attachment #8820412 - Flags: review?(gijskruitbosch+bugs) → review+
Is the patch compatible with the last ESR or two?
(In reply to James Ross from comment #3) > Is the patch compatible with the last ESR or two? Yes, nsIMutableArray has existed ~forever, and AFAICT all this code is CZ-internal at this point anyway. getClipboardTransferable only gets called from nsDragAndDrop's nsTransferable.get, which gets called from nsClipboard.js's own "read" method. nsTransferable.get converts everything from an nsI<Whatever>Array to a JS array anyway, so for consumers further down nothing changes.
Thanks.
Keywords: checkin-needed
Please rebase the trunk. unable to find 'xul/content/nsClipboard.js' for patching 1 out of 1 hunks FAILED -- saving rejects to file xul/content/nsClipboard.js.rej unable to find 'xul/content/nsDragAndDrop.js' for patching 1 out of 1 hunks FAILED -- saving rejects to file xul/content/nsDragAndDrop.js.rej patch failed, unable to continue (try -v) patch failed, rejects left in working directory errors during apply, please fix and qrefresh 1324904.patch
Keywords: checkin-needed
(In reply to Iris Hsiao [:ihsiao] from comment #6) > Please rebase the trunk. > unable to find 'xul/content/nsClipboard.js' for patching > 1 out of 1 hunks FAILED -- saving rejects to file > xul/content/nsClipboard.js.rej > unable to find 'xul/content/nsDragAndDrop.js' for patching > 1 out of 1 hunks FAILED -- saving rejects to file > xul/content/nsDragAndDrop.js.rej > patch failed, unable to continue (try -v) > patch failed, rejects left in working directory > errors during apply, please fix and qrefresh 1324904.patch WFM using `hg import bz://1324904` ? How are you importing? remote: https://hg.mozilla.org/chatzilla/rev/b7a99fda77ea3b78708b5bc635a41f6e3ab19b5c
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: