AMO warning

LouCypher · Oct 7, 2013 · af5a8f5 · af5a8f5
1 parent 4856261
commit af5a8f5
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/extension/bootstrap.js b/extension/bootstrap.js
@@ -206,7 +206,15 @@ function startup(data, reason) {
     ],
     returnType: "string",
     exec: function(args, context) {
-      let prefname = "general.useragent.locale";
+      let prefname = "general.useragent.locale";  // See AMO warning below
+      /*
+        AMO warning from add-on validator:
+        ---
+        Potentially unsafe preference branch referenced
+        Warning: Extensions should not alter preferences in this preference branch
+        ---
+        I couldn't find another way to change browser locale without using preferences
+      */
       let prefs = Services.prefs;
       let msgPrefix = "Browser language: '";
       let msgSuffix = "'";