Remove proxy.onProxyError
Categories
(WebExtensions :: General, enhancement, P3)
Tracking
(Not tracked)
People
(Reporter: eros_uk, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [proxy])
User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:56.0) Gecko/20100101 Firefox/56.0 Build ID: 20170805085114 Steps to reproduce: Further to Bug 1371879 discussion which led to renaming and addition of proxy.register & proxy.unregister, it was agreed to rename proxy.onProxyError to proxy.onError but it seems the bug was never filled. Further ref: Bug 1293751
Updated•7 years ago
|
Updated•7 years ago
|
Comment 1•6 years ago
|
||
onError added and a deprecated note in schema in bug 1409878. Probably before the next (after 60) esr we should remove the onProxyError listener.
Updated•6 years ago
|
Comment 2•5 years ago
|
||
need to remove these only?
https://searchfox.org/mozilla-central/search?q=proxy.onProxyError&case=false®exp=false&path=
It would be better to ask Shane
Comment 4•5 years ago
|
||
The use in the tests should change to proxy.onError.
The implementation in ext-proxy would be removed.
Prior to doing this, we need to go through whatever process we need to do for deprecating/removal of these.
https://wiki.mozilla.org/WebExtensions/DeprecationPolicy
We should probably start this process for onProxyError and register/unregister if someone wants to drive that.
Lets see if Mike wants to, or if we can drive that via contributor.
Comment 5•5 years ago
|
||
I plan to drive the deprecation process for both proxy.onProxyError and proxy.register/unregister.
Updated•5 years ago
|
Comment 6•5 years ago
|
||
Going back to the original (accurate) title.
Target is Firefox 71.
Updated•5 years ago
|
Comment 7•5 years ago
|
||
This will just be part of bug 1443259
Description
•