Bug 1356569 - Remove notifyObservers' last parameter when it is falsy, r=jaws.
This commit is contained in:
@@ -116,7 +116,7 @@ add_task(function* test_experiments_api() {
|
||||
|
||||
do_register_cleanup(() => {
|
||||
for (let file of [apiAddonFile, addonFile, boringAddonFile]) {
|
||||
Services.obs.notifyObservers(file, "flush-cache-entry", null);
|
||||
Services.obs.notifyObservers(file, "flush-cache-entry");
|
||||
file.remove(false);
|
||||
}
|
||||
});
|
||||
|
||||
@@ -32,6 +32,6 @@ add_task(function* test_json_parser() {
|
||||
Assert.deepEqual(extension.rawManifest, expectedManifest,
|
||||
"Manifest with correctly-filtered comments");
|
||||
|
||||
Services.obs.notifyObservers(xpi, "flush-cache-entry", null);
|
||||
Services.obs.notifyObservers(xpi, "flush-cache-entry");
|
||||
xpi.remove(false);
|
||||
});
|
||||
|
||||
@@ -55,7 +55,7 @@ add_task(function* test_embedded_webextension_utils() {
|
||||
// Remove the generated xpi file and flush the its jar cache
|
||||
// on cleanup.
|
||||
do_register_cleanup(() => {
|
||||
Services.obs.notifyObservers(fakeHybridAddonFile, "flush-cache-entry", null);
|
||||
Services.obs.notifyObservers(fakeHybridAddonFile, "flush-cache-entry");
|
||||
fakeHybridAddonFile.remove(false);
|
||||
});
|
||||
|
||||
@@ -117,7 +117,7 @@ function* createManifestErrorTestCase(id, xpi, expectedError) {
|
||||
// Remove the generated xpi file and flush the its jar cache
|
||||
// on cleanup.
|
||||
do_register_cleanup(() => {
|
||||
Services.obs.notifyObservers(xpi, "flush-cache-entry", null);
|
||||
Services.obs.notifyObservers(xpi, "flush-cache-entry");
|
||||
xpi.remove(false);
|
||||
});
|
||||
|
||||
|
||||
@@ -67,7 +67,7 @@ add_task(function* test_management_uninstall_no_prompt() {
|
||||
notEqual(addon, null, "Add-on is installed");
|
||||
extension.sendMessage("uninstall");
|
||||
yield waitForUninstalled();
|
||||
Services.obs.notifyObservers(extension.extension.file, "flush-cache-entry", null);
|
||||
Services.obs.notifyObservers(extension.extension.file, "flush-cache-entry");
|
||||
});
|
||||
|
||||
add_task(function* test_management_uninstall_prompt_uninstall() {
|
||||
@@ -97,7 +97,7 @@ add_task(function* test_management_uninstall_prompt_uninstall() {
|
||||
`The extension “${manifest.name}” is requesting to be uninstalled. What would you like to do?`);
|
||||
equal(promptService._confirmExArgs[4], "Uninstall");
|
||||
equal(promptService._confirmExArgs[5], "Keep Installed");
|
||||
Services.obs.notifyObservers(extension.extension.file, "flush-cache-entry", null);
|
||||
Services.obs.notifyObservers(extension.extension.file, "flush-cache-entry");
|
||||
});
|
||||
|
||||
add_task(function* test_management_uninstall_prompt_keep() {
|
||||
|
||||
@@ -65,7 +65,7 @@ add_task(function* test_local_cache_invalidation() {
|
||||
extension.sendMessage("set-initial");
|
||||
yield extension.awaitMessage("set-initial-done");
|
||||
|
||||
Services.obs.notifyObservers(null, "extension-invalidate-storage-cache", "");
|
||||
Services.obs.notifyObservers(null, "extension-invalidate-storage-cache");
|
||||
|
||||
extension.sendMessage("check");
|
||||
yield extension.awaitMessage("check-done");
|
||||
|
||||
@@ -14,7 +14,7 @@ function* generateAddon(data) {
|
||||
|
||||
let xpi = Extension.generateXPI(data);
|
||||
do_register_cleanup(() => {
|
||||
Services.obs.notifyObservers(xpi, "flush-cache-entry", null);
|
||||
Services.obs.notifyObservers(xpi, "flush-cache-entry");
|
||||
xpi.remove(false);
|
||||
});
|
||||
|
||||
|
||||
Reference in New Issue
Block a user