Bug 1257526 - Harmonise unsupported operation error usage; r=automatedtester draft
authorAndreas Tolfsen <ato@mozilla.com>
Thu, 17 Mar 2016 14:57:46 +0000
changeset 343884 15a36592d3548d367fb823c4e78e016d3a929762
parent 343883 03fd40e9d957901d1d60180ec8a3817f6ece4bab
child 516842 3f614daf3ed1b5d163ef01095d841ce8fcb8e51f
push id13701
push userbmo:ato@mozilla.com
push dateWed, 23 Mar 2016 13:31:33 +0000
reviewersautomatedtester
bugs1257526
milestone48.0a1
Bug 1257526 - Harmonise unsupported operation error usage; r=automatedtester Throw unsupported operation errors the same way throughout Marionette. MozReview-Commit-ID: D63gVIeX2qK
testing/marionette/driver.js
--- a/testing/marionette/driver.js
+++ b/testing/marionette/driver.js
@@ -2642,18 +2642,18 @@ GeckoDriver.prototype.getWindowSize = fu
  * Not supported on B2G. The supplied width and height values refer to
  * the window outerWidth and outerHeight values, which include scroll
  * bars, title bars, etc.
  *
  * An error will be returned if the requested window size would result
  * in the window being in the maximized state.
  */
 GeckoDriver.prototype.setWindowSize = function(cmd, resp) {
-  if (this.appName !== "Firefox") {
-    throw new UnsupportedOperationError("Not supported on mobile");
+  if (this.appName != "Firefox") {
+    throw new UnsupportedOperationError();
   }
 
   let width = parseInt(cmd.parameters.width);
   let height = parseInt(cmd.parameters.height);
 
   let win = this.getCurrentWindow();
   if (width >= win.screen.availWidth && height >= win.screen.availHeight) {
     throw new UnsupportedOperationError("Invalid requested size, cannot maximize");
@@ -2665,17 +2665,17 @@ GeckoDriver.prototype.setWindowSize = fu
 /**
  * Maximizes the user agent window as if the user pressed the maximise
  * button.
  *
  * Not Supported on B2G or Fennec.
  */
 GeckoDriver.prototype.maximizeWindow = function(cmd, resp) {
   if (this.appName != "Firefox") {
-    throw new UnsupportedOperationError("Not supported for mobile");
+    throw new UnsupportedOperationError();
   }
 
   let win = this.getCurrentWindow();
   win.moveTo(0,0);
   win.resizeTo(win.screen.availWidth, win.screen.availHeight);
 };
 
 /**