Remove RegisterProtocolHandlerInsecure deprecated operation
Categories
(Core :: DOM: Core & HTML, task)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox69 | --- | fixed |
People
(Reporter: ehsan.akhgari, Assigned: gvelchuru)
References
Details
(Whiteboard: [good first bug][mentor=ehsan])
Attachments
(1 file)
Bug 1460506 removed support for registerProtocolHandler from insecure contexts. We have not yet removed the deprecated operation warning for it. We should do that since that warning will never be triggered now.
All that is needed to fix this bug is to remove the following blocks of code essentially, this is the code that defines this deprecated operation warning code and emits it.
- https://searchfox.org/mozilla-central/rev/153172de0c5bfca31ef861bd8fc0995f44cada6a/dom/base/nsDeprecatedOperationList.h#36
- https://searchfox.org/mozilla-central/rev/153172de0c5bfca31ef861bd8fc0995f44cada6a/dom/locales/en-US/chrome/dom/dom.properties#353-354
- https://searchfox.org/mozilla-central/rev/153172de0c5bfca31ef861bd8fc0995f44cada6a/dom/base/Navigator.cpp#961-963
|
||
Updated•5 years ago
|
|
Assignee | |
Comment 1•5 years ago
|
||
I'm interested in this, can I be made assignee?
|
Reporter | |
Comment 2•5 years ago
|
||
Of course you may! Let me know if you needed any help. :-)
|
|
Assignee | |
Comment 3•5 years ago
|
||
|
|
Assignee | |
Comment 4•5 years ago
|
||
Let me know if there's anything else needed!
|
|
Reporter | |
Comment 5•5 years ago
|
||
Thanks for your patch, I submitted a try server run here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=025dbb1d22b87ef650edfc58dd1af35ac02d7d7b
Once the try job is green I will proceed to check it in for you. :-)
Pushed by eakhgari@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e6faa87de16c Remove deprecated op r=Ehsan
|
||
Comment 7•5 years ago
|
||
| bugherder | ||
Description
•