Note: There are a few cases of duplicates in user autocompletion which are being worked on.

nsIBrowserSearchEngine interface on FF24b needs a uuid bump

RESOLVED FIXED in Firefox 24

Status

()

Firefox
Search
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: WeirdAl, Assigned: Gavin)

Tracking

({addon-compat})

24 Branch
Firefox 26
addon-compat
Points:
---

Firefox Tracking Flags

(firefox24+ fixed, firefox25+ fixed, firefox26 fixed)

Details

(Whiteboard: [qa-], URL)

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
On FF24b, there was a change to nsIBrowserSearchService's methods (addEngine in particular) that didn't get a corresponding uuid change.  The latest uuid change to that interface appears to predate the method change.

I haven't checked FF25 aurora yet, or trunk.
(Reporter)

Updated

4 years ago
Blocks: 493051
tracking-firefox24: ? → +
tracking-firefox25: --- → +
Created attachment 794266 [details] [diff] [review]
uuid bump

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 493051
User impact if declined: potential for crashing binary addons
Testing completed (on m-c, etc.): N/A
Risk to taking this patch (and alternatives if risky): compatibility break for addons on beta that have already been compiled. Alternative for beta is to not take the change, in which case addons calling this method may crash.
String or IDL/UUID changes made by this patch: yes, that's the intent.
Attachment #794266 - Flags: approval-mozilla-beta?
Attachment #794266 - Flags: approval-mozilla-aurora?
(In reply to :Gavin Sharp (use gavin@gavinsharp.com for email) from comment #1)
> Created attachment 794266 [details] [diff] [review]
> uuid bump
> 
> [Approval Request Comment]
> Bug caused by (feature/regressing bug #): bug 493051
> User impact if declined: potential for crashing binary addons
> Testing completed (on m-c, etc.): N/A
> Risk to taking this patch (and alternatives if risky): compatibility break
> for addons on beta that have already been compiled. Alternative for beta is
> to not take the change, in which case addons calling this method may crash.
> String or IDL/UUID changes made by this patch: yes, that's the intent.

Do we know of any addon's/plugins that using this interface and may be impacted ? In the past we have gone with option of actually not bumping the IID post Beta 1 and since the addon testing is already completed by this time and instead did some outreach letting them know about the issue.But, :Gavin points out the risk of a potential crash in this situation :( Is taking the patch the lowest risk option here ?
The potential breakage here would happen with binary add-ons, and we never know for sure which interfaces they are using, since they are mostly hosted outside of AMO. Given that many alter search settings it's very possible that some would be affected...

I don't think there's a good solution here, but I would favor doing the uuid bump for the sake of correctness.
(In reply to Jorge Villalobos [:jorgev] from comment #3)
> The potential breakage here would happen with binary add-ons, and we never
> know for sure which interfaces they are using, since they are mostly hosted
> outside of AMO. Given that many alter search settings it's very possible
> that some would be affected...
> 
> I don't think there's a good solution here, but I would favor doing the uuid
> bump for the sake of correctness.

Thanks Jorgev, Approving the patch and I will work with you on proactively trying to contact popular addon's like we've done in the past.

Updated

4 years ago
Attachment #794266 - Flags: approval-mozilla-beta?
Attachment #794266 - Flags: approval-mozilla-beta+
Attachment #794266 - Flags: approval-mozilla-aurora?
Attachment #794266 - Flags: approval-mozilla-aurora+
Keywords: addon-compat
https://hg.mozilla.org/integration/fx-team/rev/6159d3584b37
Keywords: checkin-needed
Whiteboard: [c-n for beta/aurora]
Apparently landing this required a clobber on Windows.
https://hg.mozilla.org/mozilla-central/rev/6159d3584b37
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
https://hg.mozilla.org/releases/mozilla-aurora/rev/40abc58af0c1
https://hg.mozilla.org/releases/mozilla-beta/rev/a5c5523c4123
status-firefox24: --- → fixed
status-firefox25: --- → fixed
status-firefox26: --- → fixed
Whiteboard: [c-n for beta/aurora]
Assuming no QA needed here. Naturally we'll keep an eye out for any addon-compat issues as is our normal process. Please remove [qa-] from the whiteboard and add the verifyme keyword if this needs QA.
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.