Fix typos in code comments

jwilk · jwilk · commit 8614201b7072 · 2015-12-14T13:30:19.000+01:00
diff --git a/chromium/lru.js b/chromium/lru.js
@@ -31,7 +31,7 @@ function LRUCache (limit) {
  */
 LRUCache.prototype.put = function(key, value) {
   var entry = {key:key, value:value};
-  // Note: No protection agains replacing, and thus orphan entries. By design.
+  // Note: No protection against replacing, and thus orphan entries. By design.
   this._keymap[key] = entry;
   if (this.tail) {
     // link previous tail to the new tail (entry)
@@ -123,7 +123,7 @@ LRUCache.prototype.get = function(key, returnEntry) {
 
 /**
  * Check if <key> is in the cache without registering recent use. Feasible if
- * you do not want to chage the state of the cache, but only "peek" at it.
+ * you do not want to change the state of the cache, but only "peek" at it.
  * Returns the entry associated with <key> if found, or undefined if not found.
  */
 LRUCache.prototype.find = function(key) {
@@ -178,7 +178,7 @@ LRUCache.prototype.remove = function(key) {
 
 /** Removes all entries */
 LRUCache.prototype.removeAll = function() {
-  // This should be safe, as we never expose strong refrences to the outside
+  // This should be safe, as we never expose strong references to the outside
   this.head = this.tail = undefined;
   this.size = 0;
   this._keymap = {};
@@ -200,7 +200,7 @@ if (typeof Object.keys === 'function') {
 
 /**
  * Call `fun` for each entry. Starting with the newest entry if `desc` is a true
- * value, otherwise starts with the oldest (head) enrty and moves towards the
+ * value, otherwise starts with the oldest (head) entry and moves towards the
  * tail.
  *
  * `fun` is called with 3 arguments in the context `context`:
diff --git a/chromium/rules.js b/chromium/rules.js
@@ -300,7 +300,7 @@ RuleSets.prototype = {
   },
 
   /**
-   * Check to see if the Cookie object c meets any of our cookierule citeria for being marked as secure.
+   * Check to see if the Cookie object c meets any of our cookierule criteria for being marked as secure.
    * knownHttps is true if the context for this cookie being set is known to be https.
    * @param cookie The cookie to test
    * @param knownHttps Is the context for setting this cookie is https ?
diff --git a/src/chrome/content/code/ApplicableList.js b/src/chrome/content/code/ApplicableList.js
@@ -28,7 +28,7 @@ ApplicableList.prototype = {
 
   empty: function() {
     // Empty everything, used when toggles occur in order to ensure that if
-    // the reload fails, the resulting list is not eroneous
+    // the reload fails, the resulting list is not erroneous
     this.active = {};
     this.breaking = {}; 
     this.inactive = {};
diff --git a/src/chrome/content/code/HTTPSRules.js b/src/chrome/content/code/HTTPSRules.js
@@ -560,7 +560,7 @@ const HTTPSRules = {
         }
     } catch(e3) {
       this.log(INFO, "uri.host is explosive!");
-      try       { this.log(INFO, "(" + uri.spec + ")"); }  // happens for about: uris and soforth
+      try       { this.log(INFO, "(" + uri.spec + ")"); }  // happens for about: uris and so forth
       catch(e4) { this.log(WARN, "(and unprintable!!!!!!)"); }
     }
     return uri;
@@ -676,7 +676,7 @@ const HTTPSRules = {
   },
 
   /**
-   * Check to see if the Cookie object c meets any of our cookierule citeria
+   * Check to see if the Cookie object c meets any of our cookierule criteria
    * for being marked as secure.
    *
    * @param applicable_list {ApplicableList} an ApplicableList for record keeping
diff --git a/src/chrome/content/code/IOUtil.js b/src/chrome/content/code/IOUtil.js
@@ -14,7 +14,7 @@ const IO = {
     res = sis.read(sis.available());
     is.close();
     
-    if (charset !== null) { // use "null" if you want uncoverted data...
+    if (charset !== null) { // use "null" if you want unconverted data...
       const unicodeConverter = Cc["@mozilla.org/intl/scriptableunicodeconverter"]
         .createInstance(Ci.nsIScriptableUnicodeConverter);
       try {
diff --git a/src/components/https-everywhere.js b/src/components/https-everywhere.js
@@ -601,7 +601,7 @@ HTTPSEverywhere.prototype = {
   },
 
   maybeCleanupObservatoryPrefs: function(ssl_observatory) {
-    // Recover from a past UI processing bug that would leave the Obsevatory
+    // Recover from a past UI processing bug that would leave the Observatory
     // accidentally disabled for some users
     // https://trac.torproject.org/projects/tor/ticket/10728
     var clean = ssl_observatory.myGetBoolPref("clean_config");
@@ -852,7 +852,7 @@ function https_everywhereLog(level, str) {
   if (level >= threshold) {
     var levelName = ["", "VERB", "DBUG", "INFO", "NOTE", "WARN"][level];
     var prefix = "HTTPS Everywhere " + levelName + ": ";
-    // dump() prints to browser stdout. That's sometimes undesireable,
+    // dump() prints to browser stdout. That's sometimes undesirable,
     // so only do it when a pref is set (running from test.sh enables
     // this pref).
     if (prefs.getBoolPref("log_to_stdout")) {
diff --git a/src/components/ssl-observatory.js b/src/components/ssl-observatory.js
@@ -841,7 +841,7 @@ SSLObservatory.prototype = {
      * custom_proxy_type == "direct" is indicative of the user having selected "submit certs even if
      * Tor is not available", rather than true custom Tor proxy settings.  So in that case, there's
      * not much point probing to see if the direct proxy is actually a Tor connection, and
-     * localhost:9050 is a better bet.  People whose networks send all traffc through Tor can just
+     * localhost:9050 is a better bet.  People whose networks send all traffic through Tor can just
      * tell the Observatory to submit certs without Tor.
      */
     } else if (this.myGetBoolPref("use_custom_proxy") && !(testingForTor && custom_proxy_type == "direct")) {
@@ -924,7 +924,7 @@ SSLObservatory.prototype = {
     if (level >= threshold) {
       var levelName = ["", "VERB", "DBUG", "INFO", "NOTE", "WARN"][level];
       var prefix = "SSL Observatory " + levelName + ": ";
-      // dump() prints to browser stdout. That's sometimes undesireable,
+      // dump() prints to browser stdout. That's sometimes undesirable,
       // so only do it when a pref is set (running from test.sh enables
       // this pref).
       if (this.prefs.getBoolPref("extensions.https_everywhere.log_to_stdout")) {
diff --git a/src/defaults/preferences/preferences.js b/src/defaults/preferences/preferences.js
@@ -23,7 +23,7 @@ pref("extensions.https_everywhere.http_nowhere.orig.ocsp.required", false);
 // SSl Observatory preferences
 pref("extensions.https_everywhere._observatory.enabled",false);
 
-// "testing" currently means send unecessary fingerprints and other test-suite
+// "testing" currently means send unnecessary fingerprints and other test-suite
 // type stuff
 pref("extensions.https_everywhere._observatory.testing",false);
 
diff --git a/test/firefox/test_profile_skeleton/prefs.js b/test/firefox/test_profile_skeleton/prefs.js
@@ -8,7 +8,7 @@ user_pref("extensions.https_everywhere.LogLevel", 0);
 user_pref("extensions.https_everywhere.log_to_stdout", true);
 // Make it quicker to make manual config changes.
 user_pref("general.warnOnAboutConfig", false);
-// Minimize unnecesary requests.
+// Minimize unnecessary requests.
 user_pref("browser.safebrowsing.enabled", false);
 user_pref("browser.safebrowsing.malware.enabled", false);
 // These two preferences allow debugging the extension

Original file line number	Diff line number	Diff line change
`@@ -560,7 +560,7 @@ const HTTPSRules = {`
`560`	`560`	`}`
`561`	`561`	`} catch(e3) {`
`562`	`562`	`this.log(INFO, "uri.host is explosive!");`
`563`		`- try { this.log(INFO, "(" + uri.spec + ")"); } // happens for about: uris and soforth`
	`563`	`+ try { this.log(INFO, "(" + uri.spec + ")"); } // happens for about: uris and so forth`
`564`	`564`	`catch(e4) { this.log(WARN, "(and unprintable!!!!!!)"); }`
`565`	`565`	`}`
`566`	`566`	`return uri;`
`@@ -676,7 +676,7 @@ const HTTPSRules = {`
`676`	`676`	`},`
`677`	`677`
`678`	`678`	`/**`
`679`		`- * Check to see if the Cookie object c meets any of our cookierule citeria`
	`679`	`+ * Check to see if the Cookie object c meets any of our cookierule criteria`
`680`	`680`	`* for being marked as secure.`
`681`	`681`	`*`
`682`	`682`	`* @param applicable_list {ApplicableList} an ApplicableList for record keeping`