1277065 - HTMLElement.get_property doesn't work in "chrome" context

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Mark Banner (:standard8)]

In a browser element, we have a video element loaded in the document.

If we do:

video_element = self.marionette.find_element(By.ID, "remote-video");
video_element.get_property("paused")

This works if we're in the "content" context. However, if we've previously done a set_context("chrome"), this fails, and we have to revert to get_attribute().

For some of the Hello code, we currently have to access the browser elements via the chrome context, so this is causing more complexity than necessary.

Comment 1

[David Burns :automatedtester]

So looking at the code and what you describe... I think we have 2 separate issues. calling get_property in chrome should throw UnsupportedOperationException

e.g.

0:05.70 TEST_END: MainThread ERROR, expected PASS
Traceback (most recent call last):
  File "/Users/dburns/development/mozilla-central/testing/marionette/harness/marionette/marionette_test.py", line 344, in run
    testMethod()
  File "/Users/dburns/development/mozilla-central/testing/marionette/harness/marionette/tests/unit/test_element_state_chrome.py", line 87, in test_get
    self.assertEqual(el.get_property("id"), "textInput")
  File "/Users/dburns/development/mozilla-central/testing/marionette/client/marionette_driver/marionette.py", line 73, in get_property
    return self.marionette._send_message("getElementProperty", body, key="value")
  File "/Users/dburns/development/mozilla-central/testing/marionette/client/marionette_driver/decorators.py", line 36, in _
    return func(*args, **kwargs)
  File "/Users/dburns/development/mozilla-central/testing/marionette/client/marionette_driver/marionette.py", line 688, in _send_message
    self._handle_error(err)
  File "/Users/dburns/development/mozilla-central/testing/marionette/client/marionette_driver/marionette.py", line 721, in _handle_error
    raise errors.lookup(error)(message, stacktrace=stacktrace)
UnsupportedOperationException: UnsupportedOperationException

Comment 2

[David Burns :automatedtester]

It looks like getElementAttribute in chrome still has the conflated result of returning properties and attributes

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

This should have been actually fixed by bug 1277083. Bug 1277090 is not related here given that this is for the get_attribute() call.