Bug 1277083: Have getElementProperty return element properties in chr…

…ome. r=jgriffin

This removes the UnknowOperationError that was being thrown and returns the
property on the element that has been requested.

MozReview-Commit-ID: 2WCnBfdmit5

--HG--
extra : rebase_source : 92549bc482b3cf383bef35efa3ad10b3bcb0153e

testing/marionette/driver.js

@@ -1811,7 +1811,10 @@ GeckoDriver.prototype.getElementProperty = function*(cmd, resp) {
 
   switch (this.context) {
     case Context.CHROME:
-      throw new UnsupportedOperationError();
+      let win = this.getCurrentWindow();
+      let el = this.curBrowser.seenEls.get(id, {frame: win});
+      resp.body.value = el[name];
+      break;
 
     case Context.CONTENT:
       return this.listener.getElementProperty(id, name);

testing/marionette/harness/marionette/tests/unit/test_element_state_chrome.py

@@ -66,3 +66,22 @@ def tearDown(self):
     def test_get(self):
         el = self.marionette.execute_script("return window.document.getElementById('textInput');")
         self.assertEqual(el.get_attribute("id"), "textInput")
+
+class TestGetElementProperty(MarionetteTestCase):
+    def setUp(self):
+        MarionetteTestCase.setUp(self)
+        self.marionette.set_context("chrome")
+        self.win = self.marionette.current_window_handle
+        self.marionette.execute_script("window.open('chrome://marionette/content/test.xul', 'foo', 'chrome,centerscreen');")
+        self.marionette.switch_to_window('foo')
+        self.assertNotEqual(self.win, self.marionette.current_window_handle)
+
+    def tearDown(self):
+        self.assertNotEqual(self.win, self.marionette.current_window_handle)
+        self.marionette.execute_script("window.close();")
+        self.marionette.switch_to_window(self.win)
+        MarionetteTestCase.tearDown(self)
+
+    def test_get(self):
+        el = self.marionette.execute_script("return window.document.getElementById('textInput');")
+        self.assertEqual(el.get_property("id"), "textInput")