Bug 1246411 - Remove unreachable break statements; r?automatedtester draft
authorAndreas Tolfsen <ato@mozilla.com>
Sat, 06 Feb 2016 18:19:46 +0000
changeset 329411 ec47a18769d6e042829e5d7fd4339bf4c44d46be
parent 329410 d432fc7323535e55fa5d835c76bec3b350c85775
child 329412 c4db7079b4f950d3b8bb1698aa144664e36bc56c
push id10518
push userbmo:ato@mozilla.com
push dateSat, 06 Feb 2016 18:25:47 +0000
reviewersautomatedtester
bugs1246411
milestone47.0a1
Bug 1246411 - Remove unreachable break statements; r?automatedtester MozReview-Commit-ID: 7l0OEwIKoHS
testing/marionette/driver.js
--- a/testing/marionette/driver.js
+++ b/testing/marionette/driver.js
@@ -711,16 +711,17 @@ GeckoDriver.prototype.setUpProxy = funct
 
       case "SYSTEM":
         Preferences.set("network.proxy.type", 5);
         break;
 
       case "NOPROXY":
       default:
         Preferences.set("network.proxy.type", 0);
+        break;
     }
   } else {
     throw new InvalidArgumentError("Value of 'proxy' should be an object");
   }
 };
 
 /**
  * Log message.  Accepts user defined log-level.
@@ -2624,17 +2625,16 @@ GeckoDriver.prototype.takeScreenshot = f
       context.drawWindow(win, 0, 0, width, height, "rgb(255,255,255)", flags);
       let dataUrl = canvas.toDataURL("image/png", "");
       let data = dataUrl.substring(dataUrl.indexOf(",") + 1);
       resp.body.value = data;
       break;
 
     case Context.CONTENT:
       return this.listener.takeScreenshot(id, full, highlights);
-      break;
   }
 };
 
 /**
  * Get the current browser orientation.
  *
  * Will return one of the valid primary orientation values
  * portrait-primary, landscape-primary, portrait-secondary, or