Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: lib/typoBootstrap.js

Issue 8788183: Detect and handle the case of our own typo correction extension being installed in parallel (Closed)
Patch Set: Reverted small change in typoFixer (necessary for other project) Created Nov. 19, 2012, 3:37 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lib/main.js ('k') | lib/typoFixer.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/typoBootstrap.js
===================================================================
new file mode 100644
--- /dev/null
+++ b/lib/typoBootstrap.js
@@ -0,0 +1,79 @@
+/*
+ * This Source Code is subject to the terms of the Mozilla Public License
+ * version 2.0 (the "License"). You can obtain a copy of the License at
+ * http://mozilla.org/MPL/2.0/.
+ */
+
+/**
+ * @fileOverview Adds typo correction feature
+ */
+
+Cu.import("resource://gre/modules/AddonManager.jsm");
+
+let {TimeLine} = require("timeline");
+let {Prefs} = require("prefs");
+TimeLine.log("Done loading preferences");
Wladimir Palant 2012/11/19 17:24:23 Please remove this, it assumes that no other modul
+
+let urlfixerID = "{0fa2149e-bb2c-4ac2-a8d3-479599819475}";
+let addonListener = null;
+
+function init()
+{
+ if (!Prefs.correctTyposAsked || (Prefs.correctTyposAsked && Prefs.correctTypos))
+ {
+ AddonManager.getAddonByID(urlfixerID, function(addon)
+ {
+ startTypoCorrection(addon && addon.isActive);
+ });
+ }
+ else
+ {
+ function onPrefChange(name)
+ {
+ if (name == "correctTypos" && Prefs[name])
+ {
+ init();
+ Prefs.removeListener(onPrefChange);
+ }
+ }
+
+ Prefs.addListener(onPrefChange);
+ }
+}
+
+function startTypoCorrection(isInstalledAndEnabled)
+{
+ if (isInstalledAndEnabled)
+ require("typoFixer").detachWindowObserver();
+ else
+ require("typoFixer").attachWindowObserver();
+
+ if (!addonListener)
+ {
+ addonListener = {
+ onEnabling: function(addon, needsRestart)
+ {
+ if (addon.id == urlfixerID)
+ startTypoCorrection(true);
+ },
+ onDisabled: function(addon)
+ {
+ if (addon.id == urlfixerID)
+ startTypoCorrection(false);
+ },
+ onInstalling: function(addon, needsRestart)
+ {
+ if (addon.id == urlfixerID)
+ startTypoCorrection(true);
+ },
+ onUninstalled: function(addon)
+ {
+ if (addon.id == urlfixerID)
+ startTypoCorrection(false);
+ }
+ }
+ AddonManager.addAddonListener(addonListener);
Wladimir Palant 2012/11/19 17:24:23 This needs cleanup code - remove this listener on
+ }
+}
+
+init();
« no previous file with comments | « lib/main.js ('k') | lib/typoFixer.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld