1272494 - Use distinct chrome:// space for android, chrome://browser/ -> chrome://mobile/

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[Axel Hecht [:Pike]]

chrome:// urls aren't all that universal, one aspect of it being that fennec uses chrome://browser/ and desktop does, too. And they use the same urls, for different things.

This might have add-on compat impact, but margeret said on https://mail.mozilla.org/pipermail/mobile-firefox-dev/2016-May/002081.html that that would be OK.

Jorge, do we have a way to query mxr on addons for mobile only?

I have a local patch that I didn't actually test yet, I pushed that as part of https://treeherder.mozilla.org/#/jobs?repo=try&revision=0e82723c0237 to let try figure this out.

Comment 1

[Jorge Villalobos [:jorgev] (he/him)]

> Jorge, do we have a way to query mxr on addons for mobile only?

No, but there aren't that many Android add-ons, so they could be checked semi-manually.

Comment 2

[Axel Hecht [:Pike]]

The prior patch failed on missing chrome registry changes, which https://treeherder.mozilla.org/#/jobs?repo=try&revision=44aa1fa4e8eb has fixed.

There's a reliable failure on chrome mochitest, https://treeherder.mozilla.org/#/jobs?repo=try&revision=44aa1fa4e8eb&selectedJob=20839905, not sure how to fix that, as I don't really understand what might be broken.

Comment 3

[Axel Hecht [:Pike]]

Cleaning up local patches, I don't think this is worth it anymore.