Bug 1321516 - Remove specificationLevel capability; r?automatedtester draft
authorAndreas Tolfsen <ato@mozilla.com>
Sat, 10 Dec 2016 17:48:43 -1000
changeset 448592 efc3154a0445baf619fbd8ad4c077042c163b3ac
parent 447934 d8f63b2935af0915a6a24b3ea8e27d9a09f66416
child 448593 1aed65fe80ef18b9b98ed43cf4022148b06e8997
push id38373
push userbmo:ato@mozilla.com
push dateSun, 11 Dec 2016 03:55:07 +0000
reviewersautomatedtester
bugs1321516
milestone53.0a1
Bug 1321516 - Remove specificationLevel capability; r?automatedtester MozReview-Commit-ID: BrJIOpdyj5E
testing/marionette/driver.js
testing/marionette/harness/marionette/tests/unit/test_capabilities.py
testing/marionette/listener.js
--- a/testing/marionette/driver.js
+++ b/testing/marionette/driver.js
@@ -154,17 +154,16 @@ this.GeckoDriver = function(appName, ser
     "acceptInsecureCerts": !this.secureTLS,
 
     // supported features
     "raisesAccessibilityExceptions": false,
     "rotatable": this.appName == "B2G",
     "proxy": {},
 
     // proprietary extensions
-    "specificationLevel": 0,
     "moz:processID": Services.appinfo.processID,
     "moz:profile": Services.dirsvc.get("ProfD", Ci.nsIFile).path,
   };
 
   this.mm = globalMessageManager;
   this.listener = proxy.toListener(() => this.mm, this.sendAsync.bind(this));
 
   // always keep weak reference to current dialogue
--- a/testing/marionette/harness/marionette/tests/unit/test_capabilities.py
+++ b/testing/marionette/harness/marionette/tests/unit/test_capabilities.py
@@ -19,23 +19,21 @@ class TestCapabilities(MarionetteTestCas
             self.os_version = self.marionette.execute_script(
                 "return Services.sysinfo.getProperty('version')")
 
     def test_mandates_capabilities(self):
         self.assertIn("browserName", self.caps)
         self.assertIn("browserVersion", self.caps)
         self.assertIn("platformName", self.caps)
         self.assertIn("platformVersion", self.caps)
-        self.assertIn("specificationLevel", self.caps)
 
         self.assertEqual(self.caps["browserName"], self.appinfo["name"].lower())
         self.assertEqual(self.caps["browserVersion"], self.appinfo["version"])
         self.assertEqual(self.caps["platformName"], self.os_name)
         self.assertEqual(self.caps["platformVersion"], self.os_version)
-        self.assertEqual(self.caps["specificationLevel"], 0)
 
     def test_supported_features(self):
         self.assertIn("rotatable", self.caps)
         self.assertIn("acceptInsecureCerts", self.caps)
         self.assertFalse(self.caps["acceptInsecureCerts"])
 
     def test_additional_capabilities(self):
         self.assertIn("moz:processID", self.caps)
@@ -49,22 +47,16 @@ class TestCapabilities(MarionetteTestCas
 
     def test_we_can_pass_in_capabilities_on_session_start(self):
         self.marionette.delete_session()
         capabilities = {"desiredCapabilities": {"somethingAwesome": "cake"}}
         self.marionette.start_session(capabilities)
         caps = self.marionette.session_capabilities
         self.assertIn("somethingAwesome", caps)
 
-    def test_set_specification_level(self):
-        self.marionette.delete_session()
-        self.marionette.start_session({"specificationLevel": 1})
-        caps = self.marionette.session_capabilities
-        self.assertEqual(1, caps["specificationLevel"])
-
     def test_we_dont_overwrite_server_capabilities(self):
         self.marionette.delete_session()
         capabilities = {"desiredCapabilities": {"browserName": "ChocolateCake"}}
         self.marionette.start_session(capabilities)
         caps = self.marionette.session_capabilities
         self.assertEqual(caps["browserName"], self.appinfo["name"].lower(),
                          "This should have appname not ChocolateCake.")
 
--- a/testing/marionette/listener.js
+++ b/testing/marionette/listener.js
@@ -1153,18 +1153,17 @@ function getActiveElement() {
  *
  * @param {WebElement} id
  *     Reference to the web element to click.
  */
 function clickElement(id) {
   let el = seenEls.get(id, curContainer);
   return interaction.clickElement(
       el,
-      !!capabilities.raisesAccessibilityExceptions,
-      capabilities.specificationLevel >= 1);
+      !!capabilities.raisesAccessibilityExceptions);
 }
 
 function getElementAttribute(id, name) {
   let el = seenEls.get(id, curContainer);
   if (element.isBooleanAttribute(el, name)) {
     if (el.hasAttribute(name)) {
       return "true";
     } else {