Bug 1517505 - Remove app/feature and bootstrap exceptions from all_files_referenced test r=Gijs

Differential Revision: https://phabricator.services.mozilla.com/D15783

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Dhyey Thakore 2019-01-08 15:25:50 +00:00
Родитель a0640935f5
Коммит 1208511f7b
1 изменённых файлов: 0 добавлений и 27 удалений

Просмотреть файл

@ -342,13 +342,6 @@ function addCodeReference(url, fromURI) {
if (ref === null)
return;
} else {
// Mark any file referenced by a 'features' bootstrap.js file as
// unconditionally referenced. The features folder is only in
// resource://app/ for non-packaged builds.
if (/resource:\/\/app\/features\/[^/]+\/bootstrap\.js/.test(from)) {
gReferencesFromCode.set(url, null);
return;
}
ref = new Set();
gReferencesFromCode.set(url, ref);
}
@ -680,9 +673,6 @@ add_task(async function checkAllTheFiles() {
if (refs === null)
return false;
for (let ref of refs) {
if (ref.endsWith("!/bootstrap.js"))
return false;
if (isDevtools) {
if (ref.startsWith("resource://app/components/") ||
(file.startsWith("chrome://") && ref.startsWith("resource://")))
@ -755,23 +745,6 @@ add_task(async function checkAllTheFiles() {
});
}
unreferencedFiles = unreferencedFiles.filter(file => {
// resource://app/features/ will only contain .xpi files when the test runs
// on a packaged build, so the following two exceptions only matter when
// running the test on a local non-packaged build.
if (/resource:\/\/app\/features\/[^/]+\/bootstrap\.js/.test(file)) {
info("not reporting feature boostrap file: " + file);
return false;
}
// Bug 1351892 - can stop shipping these?
if (/resource:\/\/app\/features\/[^/]+\/chrome\/skin\//.test(file)) {
info("not reporting feature skin file that may be for another platform: " + file);
return false;
}
return true;
});
is(unreferencedFiles.length, 0, "there should be no unreferenced files");
for (let file of unreferencedFiles) {
let refs = gReferencesFromCode.get(file);