Bug 1301315 - Add support for chrome_settings_overrides/search_provider. r?mixedpuppy
MozReview-Commit-ID: HFz6ymSjZ8t
--- a/browser/components/extensions/ext-chrome-settings-overrides.js
+++ b/browser/components/extensions/ext-chrome-settings-overrides.js
@@ -2,25 +2,69 @@
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
"use strict";
XPCOMUtils.defineLazyModuleGetter(this, "ExtensionPreferencesManager",
"resource://gre/modules/ExtensionPreferencesManager.jsm");
+var searchEngineMap = new WeakMap();
+
this.chrome_settings_overrides = class extends ExtensionAPI {
onManifestEntry(entryName) {
let {extension} = this;
let {manifest} = extension;
if (manifest.chrome_settings_overrides.homepage) {
ExtensionPreferencesManager.setSetting(extension, "homepage_override",
manifest.chrome_settings_overrides.homepage);
}
+ if (manifest.chrome_settings_overrides.search_provider) {
+ let searchProvider = manifest.chrome_settings_overrides.search_provider;
+ try {
+ Services.search.addEngineWithDetails(searchProvider.name.trim(),
+ searchProvider.favicon_url,
+ searchProvider.keyword, null,
+ "GET", searchProvider.search_url,
+ extension.id);
+ let engine = Services.search.getEngineByName(searchProvider.name.trim());
+ searchEngineMap.set(extension, engine);
+ } catch (e) {
+ // It's possible we got restarted but we don't have a reference to our
+ // engine anymore. Attempt to find it.
+ let engine = Services.search.getEngineByName(searchProvider.name.trim());
+ if (engine) {
+ let submission = engine.getSubmission("searchTerms");
+ // This is a little hacky, but there's no way to get at
+ // the original search URL of the engine.
+ if (submission.uri.spec.replace("searchTerms", "") ==
+ searchProvider.search_url.replace("{searchTerms}", "")) {
+ searchEngineMap.set(extension, engine);
+ return;
+ }
+ }
+ Components.utils.reportError(e);
+ }
+ }
+ }
+ onShutdown(reason) {
+ let {extension} = this;
+
+ if (extension.manifest.chrome_settings_overrides.search_provider) {
+ let engine = searchEngineMap.get(extension);
+ searchEngineMap.delete(extension);
+ if (engine) {
+ try {
+ Services.search.removeEngine(engine);
+ } catch (e) {
+ Components.utils.reportError(e);
+ }
+ }
+ }
}
};
ExtensionPreferencesManager.addSetting("homepage_override", {
prefNames: [
"browser.startup.homepage",
],
setCallback(value) {
--- a/browser/components/extensions/schemas/chrome_settings_overrides.json
+++ b/browser/components/extensions/schemas/chrome_settings_overrides.json
@@ -10,16 +10,103 @@
"optional": true,
"additionalProperties": { "$ref": "UnrecognizedProperty" },
"properties": {
"homepage": {
"type": "string",
"format": "relativeUrl",
"optional": true,
"preprocess": "localize"
+ },
+ "search_provider": {
+ "type": "object",
+ "optional": true,
+ "additionalProperties": { "$ref": "UnrecognizedProperty" },
+ "properties": {
+ "name": {
+ "type": "string",
+ "preprocess": "localize"
+ },
+ "keyword": {
+ "type": "string",
+ "optional": true,
+ "preprocess": "localize"
+ },
+ "search_url": {
+ "type": "string",
+ "format": "url",
+ "pattern": "^https://.*$",
+ "preprocess": "localize"
+ },
+ "favicon_url": {
+ "type": "string",
+ "optional": true,
+ "format": "url",
+ "preprocess": "localize"
+ },
+ "suggest_url": {
+ "type": "string",
+ "optional": true,
+ "format": "url",
+ "preprocess": "localize",
+ "deprecated": "Unsupported on Firefox at this time."
+ },
+ "instant_url": {
+ "type": "string",
+ "optional": true,
+ "format": "url",
+ "preprocess": "localize",
+ "deprecated": "Unsupported on Firefox at this time."
+ },
+ "image_url": {
+ "type": "string",
+ "optional": true,
+ "format": "url",
+ "preprocess": "localize",
+ "deprecated": "Unsupported on Firefox at this time."
+ },
+ "search_url_post_params": {
+ "type": "string",
+ "optional": true,
+ "preprocess": "localize",
+ "deprecated": "Unsupported on Firefox at this time."
+ },
+ "instant_url_post_params": {
+ "type": "string",
+ "optional": true,
+ "preprocess": "localize",
+ "deprecated": "Unsupported on Firefox at this time."
+ },
+ "image_url_post_params": {
+ "type": "string",
+ "optional": true,
+ "preprocess": "localize",
+ "deprecated": "Unsupported on Firefox at this time."
+ },
+ "alternate_urls": {
+ "type": "array",
+ "items": {
+ "type": "string",
+ "format": "url",
+ "preprocess": "localize"
+ },
+ "optional": true,
+ "deprecated": "Unsupported on Firefox at this time."
+ },
+ "prepopulated_id": {
+ "type": "integer",
+ "optional": true,
+ "deprecated": "Unsupported on Firefox."
+ },
+ "is_default": {
+ "type": "boolean",
+ "optional": true,
+ "deprecated": "Unsupported on Firefox at this time."
+ }
+ }
}
}
}
}
}
]
}
]
--- a/browser/components/extensions/test/browser/browser-common.ini
+++ b/browser/components/extensions/test/browser/browser-common.ini
@@ -84,16 +84,17 @@ skip-if = debug || asan # Bug 1354681
[browser_ext_runtime_openOptionsPage_uninstall.js]
[browser_ext_runtime_setUninstallURL.js]
[browser_ext_sessions_forgetClosedTab.js]
[browser_ext_sessions_forgetClosedWindow.js]
[browser_ext_sessions_getRecentlyClosed.js]
[browser_ext_sessions_getRecentlyClosed_private.js]
[browser_ext_sessions_getRecentlyClosed_tabs.js]
[browser_ext_sessions_restore.js]
+[browser_ext_settings_overrides_search.js]
[browser_ext_sidebarAction.js]
[browser_ext_sidebarAction_context.js]
[browser_ext_sidebarAction_tabs.js]
[browser_ext_sidebarAction_windows.js]
[browser_ext_simple.js]
[browser_ext_tab_runtimeConnect.js]
[browser_ext_tabs_audio.js]
[browser_ext_tabs_captureVisibleTab.js]
new file mode 100644
--- /dev/null
+++ b/browser/components/extensions/test/browser/browser_ext_settings_overrides_search.js
@@ -0,0 +1,81 @@
+/* -*- Mode: indent-tabs-mode: nil; js-indent-level: 2 -*- */
+/* vim: set sts=2 sw=2 et tw=80: */
+
+"use strict";
+
+add_task(function* test_extension_adding_engine() {
+ let ext1 = ExtensionTestUtils.loadExtension({
+ manifest: {
+ "chrome_settings_overrides": {
+ "search_provider": {
+ "name": "MozSearch",
+ "keyword": "MozSearch",
+ "search_url": "https://example.com/?q={searchTerms}"
+ }
+ },
+ }
+ });
+
+ yield ext1.startup();
+
+ let engine = Services.search.getEngineByName("MozSearch");
+ ok(engine, "Engine should exist.");
+
+ yield ext1.unload();
+
+ engine = Services.search.getEngineByName("MozSearch");
+ ok(!engine, "Engine should not exist");
+});
+
+add_task(function* test_extension_adding_engine_with_spaces() {
+ let ext1 = ExtensionTestUtils.loadExtension({
+ manifest: {
+ "chrome_settings_overrides": {
+ "search_provider": {
+ "name": "MozSearch ",
+ "keyword": "MozSearch",
+ "search_url": "https://example.com/?q={searchTerms}"
+ }
+ },
+ }
+ });
+
+ yield ext1.startup();
+
+ let engine = Services.search.getEngineByName("MozSearch");
+ ok(engine, "Engine should exist.");
+
+ yield ext1.unload();
+
+ engine = Services.search.getEngineByName("MozSearch");
+ ok(!engine, "Engine should not exist");
+});
+
+add_task(function* test_extension_finding_existing_engine() {
+ let ext1 = ExtensionTestUtils.loadExtension({
+ manifest: {
+ "chrome_settings_overrides": {
+ "search_provider": {
+ "name": "MozSearch",
+ "keyword": "MozSearch",
+ "search_url": "https://example.com/?q={searchTerms}"
+ }
+ },
+ }
+ });
+
+ Services.search.addEngineWithDetails("MozSearch",
+ null,
+ "MozSearch", null,
+ "GET", "https://example.com/?q={searchTerms}");
+
+ yield ext1.startup();
+
+ let engine = Services.search.getEngineByName("MozSearch");
+ ok(engine, "Engine should exist.");
+
+ yield ext1.unload();
+
+ engine = Services.search.getEngineByName("MozSearch");
+ ok(!engine, "Engine should not exist");
+});