Fix the approach used in ieFirstRun.js

html/static/js/ieFirstRun.js

 var AdblockPlus = (function()
 {
-  var scopes = {
-    prefs: {

[Felix Dahlke, 2014/07/04 13:40:32]

Opening braces of multi-line object literals should go on their own line after
:. After = too actually, we only don't do it when it wouldn't work (e.g. after
"return").

-      Prefs: { documentation_link: "" }

[Felix Dahlke, 2014/07/04 13:40:32]

No space after "{" or before "}" for object literals, by convention.

+  var scopes =
+  {
+    prefs:
+    {
+      Prefs: {documentation_link: ""}
     },
-    utils: {
-        Utils: { appLocale: "" }

[Felix Dahlke, 2014/07/04 13:40:32]

Indentation should be 2 spaces per level.

+    utils:
+    {
+      Utils: {appLocale: ""}
     },
-    filterClasses: {
-        Filter: { 
-            fromText: { 
-                matches: function(param) { return true } 
-            }
+    filterClasses:
+    {
+      Filter:
+      {
+        fromText:
+        {
+          matches: function(param) {return true}
         }
-    }    
-  }; 
+      }
+    }
+  };
 
-  var result = {
-    require: function(module) {
+  var result = 
+  {
+    require: function(module)
+    {
       return scopes[module];
     }
   };
 
   window.addEventListener("load", function()
   {
-    result.getMessage = function(section, param) {
-            return Settings.GetMessage(section, param);

[Felix Dahlke, 2014/07/04 13:40:32]

Indentation is off here.

-        }         

[Felix Dahlke, 2014/07/04 13:40:32]

Is that trailing whitespace?

+    result.getMessage = function(section, param) 
+    {
+      return Settings.GetMessage(section, param);
+    }
     scopes.prefs.Prefs.documentation_link = Settings.GetDocumentationLink();
     scopes.utils.Utils.appLocale = Settings.GetAppLocale();
   }, false);
 
   return result;
 })();