Closed Bug 1065574 Opened 5 years ago Closed 5 years ago
As far as we can tell from mxr.mozilla.org/addons no addon actually passes a handler to setUserData.
Attachment #8487403 - Flags: review?(bugs)
Comment on attachment 8487403 [details] [diff] [review] UserDataHandler bz, peterv, does the .webidl change look ok to you?
...it shouldn't be too regression risky based on https://mxr.mozilla.org/addons/search?string=setUserData
Comment on attachment 8487403 [details] [diff] [review] UserDataHandler The .webidl bit looks great to me.
Attachment #8487403 - Flags: feedback?(bzbarsky) → feedback+
Comment on attachment 8487403 [details] [diff] [review] UserDataHandler You need to update uuid of nsIDOMNode and all the interfaces inheriting it. sfink's tool from https://developer.mozilla.org/en-US/docs/Generating_GUIDs#Perl might be useful for that.
Attachment #8487403 - Flags: review?(bugs) → review+
Or "mach update-uuids".
Oh, a use case for mach! :)
Status: NEW → ASSIGNED
OS: Linux → All
QA Contact: evilpies
Hardware: x86_64 → All
I assumed that f+ from bz is enough. https://tbpl.mozilla.org/?tree=Try&rev=118020b19ddd https://hg.mozilla.org/integration/mozilla-inbound/rev/9bba47e3ddf4
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in before you can comment on or make changes to this bug.