1601122 - Remove chrome "redirects" and always use the full path to a resource (i.e. use chrome://global/skin/global.css instead of chrome://global/skin/)

Status: NEW

(Core :: XUL, task, P3)

Description

[Brian Grinstead [:bgrins]]

We've noticed this doesn't work with documents anymore as they move away from .xul, and I'm not aware of a current benefit to using it instead of the full chrome path (and it can make grepping for file references harder).

Here's a test that demonstrates typical behavior: https://searchfox.org/mozilla-central/source/chrome/test/unit/test_bug564667.js.

Here are some places that may be using the feature (replacing chrome://global/skin/ will go a long way towards removing all consumers of this):

• https://searchfox.org/mozilla-central/search?q=%3D%5C%22chrome.*%5C%2F.*%2F%5C%22&case=false&regexp=true&path=
• https://searchfox.org/mozilla-central/search?q=%5C%28%5C%22chrome.*%5C%2F.*%2F%5C%22&regexp=true&path=

Comment 1

[Brian Grinstead [:bgrins]]

Here's the code that handles this: https://searchfox.org/mozilla-central/rev/8bc24752246aeac8a9aed566cf1caccf88d97d11/chrome/nsChromeRegistry.cpp#180-197.

Comment 2

[Brian Grinstead [:bgrins]]

There's also code in browser_all_files_referenced.js that emulates this behavior: https://searchfox.org/mozilla-central/rev/8bc24752246aeac8a9aed566cf1caccf88d97d11/browser/base/content/test/static/browser_all_files_referenced.js#977-979