Fix YUI Compressor incompatibilities and in so doing improve general …

…JavaScript correctness (#2452)

gulpfile.js

@@ -51,7 +51,8 @@ var webpackModule = {
       loader: 'babel',
       query: {
         cacheDirectory: true,
-        presets: ['es2015']
+        presets: ['es2015'],
+        plugins: ['transform-es3-property-literals', 'transform-es3-member-expression-literals']
       }
     }
   ],

lib/network/modules/components/shared/Label.js

@@ -519,63 +519,63 @@ class Label {
       }
     }
     while (s.position < text.length) {
-      let char = text.charAt(s.position);
-      if (/[ \t]/.test(char)) {
+      let ch = text.charAt(s.position);
+      if (/[ \t]/.test(ch)) {
         if (!s.mono) {
           s.spacing = true;
         } else {
-          s.add(char);
+          s.add(ch);
         }
         s.beginable = true
-      } else if (/\\/.test(char)) {
+      } else if (/\\/.test(ch)) {
         if (s.position < text.length+1) {
           s.position++;
-          char = text.charAt(s.position);
-          if (/ \t/.test(char)) {
+          ch = text.charAt(s.position);
+          if (/ \t/.test(ch)) {
             s.spacing = true;
           } else {
-            s.add(char);
+            s.add(ch);
             s.beginable = false;
           }
         }
-      } else if (!s.mono && !s.bold && (s.beginable || s.spacing) && /\*/.test(char)) {
+      } else if (!s.mono && !s.bold && (s.beginable || s.spacing) && /\*/.test(ch)) {
         s.emitBlock();
         s.bold = true;
         s.modStack.unshift("bold");
-      } else if (!s.mono && !s.ital && (s.beginable || s.spacing) && /\_/.test(char)) {
+      } else if (!s.mono && !s.ital && (s.beginable || s.spacing) && /\_/.test(ch)) {
         s.emitBlock();
         s.ital = true;
         s.modStack.unshift("ital");
-      } else if (!s.mono && (s.beginable || s.spacing) && /`/.test(char)) {
+      } else if (!s.mono && (s.beginable || s.spacing) && /`/.test(ch)) {
         s.emitBlock();
         s.mono = true;
         s.modStack.unshift("mono");
-      } else if (!s.mono && (s.mod() === "bold") && /\*/.test(char)) {
+      } else if (!s.mono && (s.mod() === "bold") && /\*/.test(ch)) {
         if ((s.position === text.length-1) || /[.,_` \t\n]/.test(text.charAt(s.position+1))) {
           s.emitBlock();
           s.bold = false;
           s.modStack.shift();
         } else {
-          s.add(char);
+          s.add(ch);
         }
-      } else if (!s.mono && (s.mod() === "ital") && /\_/.test(char)) {
+      } else if (!s.mono && (s.mod() === "ital") && /\_/.test(ch)) {
         if ((s.position === text.length-1) || /[.,*` \t\n]/.test(text.charAt(s.position+1))) {
           s.emitBlock();
           s.ital = false;
           s.modStack.shift();
         } else {
-          s.add(char);
+          s.add(ch);
         }
-      } else if (s.mono && (s.mod() === "mono") && /`/.test(char)) {
+      } else if (s.mono && (s.mod() === "mono") && /`/.test(ch)) {
         if ((s.position === text.length-1) || (/[.,*_ \t\n]/.test(text.charAt(s.position+1)))) {
           s.emitBlock();
           s.mono = false;
           s.modStack.shift();
         } else {
-          s.add(char);
+          s.add(ch);
         }
       } else {
-        s.add(char);
+        s.add(ch);
         s.beginable = false;
       }
       s.position++
@@ -636,14 +636,14 @@ class Label {
       }
     }
     while (s.position < text.length) {
-      let char = text.charAt(s.position);
-      if (/[ \t]/.test(char)) {
+      let ch = text.charAt(s.position);
+      if (/[ \t]/.test(ch)) {
         if (!s.mono) {
           s.spacing = true;
         } else {
-          s.add(char);
+          s.add(ch);
         }
-      } else if (/</.test(char)) {
+      } else if (/</.test(ch)) {
         if (!s.mono && !s.bold && /<b>/.test(text.substr(s.position,3))) {
           s.emitBlock();
           s.bold = true;
@@ -675,9 +675,9 @@ class Label {
           s.modStack.shift();
           s.position += 6;
         } else {
-          s.add(char);
+          s.add(ch);
         }
-      } else if (/&/.test(char)) {
+      } else if (/&/.test(ch)) {
         if (/&lt;/.test(text.substr(s.position,4))) {
           s.add("<");
           s.position += 3;
@@ -688,7 +688,7 @@ class Label {
           s.add("&");
         }
       } else {
-        s.add(char);
+        s.add(ch);
       }
       s.position++
     }