Closed Bug 1404277 Opened 7 years ago Closed 7 years ago

element.isXULElement() retrieves namespaceURI via attribute but not property

Categories

(Remote Protocol :: Marionette, enhancement, P1)

Product:
Remote Protocol
Component:
Marionette
Version:
57 Branch
Type:
enhancement

Tracking

(firefox58 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla58
Tracking Flags:
Tracking Status
firefox58 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

Bug 1404277 - Retrieve namespaceURI as property and not attribute.
(deleted), text/x-review-board-request
automatedtester
: review+
ato
: review+
Details
As MDN explains: https://developer.mozilla.org/en-US/docs/Web/API/CSSNamespaceRule/namespaceURI "The CSSNamespaceRule.namespaceURI read-only property returns a DOMString containing the text of the URI of the given namespace." Not sure why we are trying to retrieve it via getAttribute().
I assume this is a left-over from the work on bug 1272653 given that the new Selenium atoms no longer conflate attributes and properties.
Marking as P1 because it blocks us from upgrading the Selenium atoms, which seem to be necessary to fix other known bugs.
Blocks: webdriver, webdriver-chrome
Priority: -- → P1
Attachment #8913618 - Flags: review?(ato)
Comment on attachment 8913618 [details] Bug 1404277 - Retrieve namespaceURI as property and not attribute. https://reviewboard.mozilla.org/r/185010/#review190288 Thanks for fixing this. I actually have a patch for this locally that I hadn’t upstreamed.
Attachment #8913618 - Flags: review?(ato) → review+
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/65a1e4064a52 Retrieve namespaceURI as property and not attribute. r=ato
https://hg.mozilla.org/mozilla-central/rev/65a1e4064a52
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox58: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Comment on attachment 8913618 [details] Bug 1404277 - Retrieve namespaceURI as property and not attribute. https://reviewboard.mozilla.org/r/185010/#review190736
Attachment #8913618 - Flags: review?(dburns) → review+
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: