Open Bug 1053364 Opened 5 years ago Updated 4 years ago

Don't warn about nsIProtocolProxyService, nsIProxyAutoConfig, or newProxiedChannel for addons explicitly supporting current application versions

Categories

(addons.mozilla.org Graveyard :: Add-on Validation, defect, P4)

defect

Tracking

(Not tracked)

REOPENED

People

(Reporter: davemgarrett, Unassigned)

References

Details

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

The addon validator flags usage of nsIProtocolProxyService, nsIProxyAutoConfig, or newProxiedChannel (bug 795168) due to the sync API being removed in favor of async only (bug 769764).

However, the detection isn't particularly smart. I'm still getting notices about this when I upload new versions, and an AMO editor actually mentioned it to me in an email. I think it's long past time to quash these.

What I propose is only flagging these API changes in the validator if the addon's install.rdf does not explicitly set a max supported application version that is current. So, if it has Firefox 31 (current ESR), or equivalent other application version, or later in its max version, then this warning would not apply any longer.

Alternatively, the warning could just be retired altogether, but I don't see a reason not to keep it in there for people updating particularly old addons.
No longer depends on: 769764
Whiteboard: [fx18]
Thanks for filing this.  Due to resource constraints we are closing bugs which we won't realistically be able to fix.  If you have a patch that applies to this bug please reopen.

For more info see http://micropipes.com/blog/2014/09/24/the-great-add-on-bug-triage/
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
Reopening. This is just routine validator updating. The warning just needs to be updated. At bare minimum, just remove it, as it's not needed anymore.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Thanks for the info.  Removing these sounds like a good thing to do
Priority: -- → P4
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.