1499092 - Move registerProtocolHandler code out of browser/components/feeds/

Status: RESOLVED FIXED

(Firefox :: General, enhancement, P3)

Description

[:Gijs (he/him)]

It doesn't really have anything to do with feeds.

I think ideally we should be able to push more of the implementation down into DOM code, and have the DOM code explicitly ask the parent process for permission. Then we can get rid of the content-process side JS that currently handles forwarding the request to frontend code, which will also make it easier to share code with mobile / geckoview.

Ideally the protocol handling backend stuff would also be handled in DOM code, though I'm not sure if that's feasible today.

Comment 1

[:Gijs (he/him)]

To be clear, this should also move/rename the strings from `subscribe.properties` ( https://searchfox.org/mozilla-central/source/browser/locales/en-US/chrome/browser/feeds/subscribe.properties )

Comment 2

[:Gijs (he/him)]

Might be able to just do this as part of fxperf - removing all the content process JS machinery here should improve content process size + startup stuff.

Comment 3

[:Gijs (he/him)]

Attachment: Bug 1499092 - move bulk of registerProtocolHandler checks into compiled code so we don't need a dedicated component in the child process, r?nika!

Comment 4

[:Gijs (he/him)]

Attachment: Bug 1499092 - rename/move code from 'feeds' and 'web content' to more appropriate/obvious versions for registerProtocolHandler, r?nika!

Depends on D13697

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b51f62289b9a
move bulk of registerProtocolHandler checks into compiled code so we don't need a dedicated component in the child process, r=nika
https://hg.mozilla.org/integration/autoland/rev/13078c094de9
rename/move code from 'feeds' and 'web content' to more appropriate/obvious versions for registerProtocolHandler, r=nika

Comment 6

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/b51f62289b9a
https://hg.mozilla.org/mozilla-central/rev/13078c094de9