Bug 1571567 - Fix no-fallthrough errors in /dom. r=smaug

Differential Revision: https://phabricator.services.mozilla.com/D40874

--HG--
extra : moz-landing-system : lando

dom/base/test/iframe_bug976673.html

@@ -15,6 +15,7 @@
             input.blur();
             input.value = "";
             input.focus();
+          // fall through
           case "check":
             aEvent.source.postMessage("input-value: " + input.value, "*");
             break;

dom/base/test/unit/head_xml.js

@@ -103,6 +103,7 @@ function do_check_equiv(dom1, dom2) {
     case Node.PROCESSING_INSTRUCTION_NODE:
       Assert.equal(dom1.target, dom2.target);
       Assert.equal(dom1.data, dom2.data);
+    // fall through
     case Node.TEXT_NODE:
     case Node.CDATA_SECTION_NODE:
     case Node.COMMENT_NODE:

dom/browser-element/BrowserElementChildPreload.js

@@ -1171,6 +1171,7 @@ BrowserElementChild.prototype = {
           case Cr.NS_BINDING_ABORTED:
           // Ignoring NS_BINDING_ABORTED, which is set when loading page is
           // stopped.
+          // fall through
           case Cr.NS_ERROR_PARSED_DATA_CACHED:
             return;
 

dom/media/webaudio/test/test_ScriptProcessorCollected1.html

@@ -47,6 +47,7 @@ testProcessor.onaudioprocess = function(event) {
     referenceProcessor.onaudioprocess = event.target.onaudioprocess;
     referenceProcessor = null;
     event.target.onaudioprocess = null;
+    break;
   case 2:
     // There are no references to testProcessor and so GC can begin.
     SpecialPowers.forceGC();

dom/notification/NotificationStorage.jsm

@@ -180,6 +180,7 @@ NotificationStorage.prototype = {
         } catch (e) {
           debug("Error calling callback done: " + e);
         }
+        break;
       case kMessageNotificationSaveKo:
       case kMessageNotificationDeleteKo:
         if (DEBUG) {

dom/plugins/test/mochitest/test_npruntime_npninvoke.html

@@ -128,6 +128,7 @@
         case 5:
           result = plugin.npnInvokeTest(test[0][0], test[0][1], test[0][2],
               test[0][3], test[0][4]);
+          break;
         case 6:
           result = plugin.npnInvokeTest(test[0][0], test[0][1], test[0][2],
               test[0][3], test[0][4], test[0][5]);

dom/tests/mochitest/localstorage/interOriginTest.js

@@ -16,9 +16,8 @@ function onMessageReceived(event) {
         break;
       }
 
-    // Just fall through...
-
     // Indication of successfully finished step of a test
+    // fall through
     case "perf":
       if (event.data == "perf") {
         doStep();

dom/tests/mochitest/localstorage/interOriginTest2.js

@@ -22,9 +22,8 @@ function onMessageReceived(event) {
         break;
       }
 
-    // Just fall through...
-
     // Indication of successfully finished step of a test
+    // Just fall through...
     case "perf":
       if (callMasterFrame) {
         masterFrame.postMessage("step", masterFrameOrigin);

dom/tests/mochitest/localstorage/test_localStorageQuotaPrivateBrowsing_perwindowpb.html

@@ -142,8 +142,8 @@ function onMessageReceived(event, aWindow) {
     case "frame loaded":
       if (--slaveLoadsPending)
         break;
-      // Just fall through...
     // Indication of successfully finished step of a test
+    // Just fall through...
     case "perf":
       // postMessage should send to the slaveOrigin. However with the addition of private
       // browsing flags in origin attributes this will cause postMessage to fail. The origin of this

dom/tests/mochitest/sessionstorage/interOriginTest.js

@@ -17,9 +17,8 @@ function onMessageReceived(event) {
         break;
       }
 
-    // Just fall through...
-
     // Indication of successfully finished step of a test
+    // Just fall through...
     case "perf":
       // We called doStep before the frame was load
       if (event.data == "perf") {

dom/tests/mochitest/storageevent/interOriginTest2.js

@@ -21,9 +21,8 @@ function onMessageReceived(event) {
         break;
       }
 
-    // Just fall through...
-
     // Indication of successfully finished step of a test
+    // Just fall through...
     case "perf":
       if (callMasterFrame) {
         masterFrame.postMessage("step", "*");