[B2G] HTMLElement does not have size available

RESOLVED FIXED in mozilla19

Status

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: zcampbell, Assigned: automatedtester)

Tracking

unspecified
mozilla19
x86_64
Windows 7
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Reporter

Description

7 years ago
HTMLElement on B2G Marionette does not have the size property available yet and is a useful property.
Assignee

Updated

7 years ago
Assignee: nobody → dburns
Assignee

Comment 1

7 years ago
Attachment #672273 - Flags: review?(mdas)
Assignee

Comment 2

7 years ago
noticed issues with patch so have corrected
Attachment #672273 - Attachment is obsolete: true
Attachment #672273 - Flags: review?(mdas)
Attachment #673046 - Flags: review?(mdas)
Comment on attachment 673046 [details] [diff] [review]
get element size

Review of attachment 673046 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm! just clear that whitespace and you're good to go, thanks!

::: testing/marionette/marionette-actors.js
@@ +1340,5 @@
> +  getElementSize: function MDA_getElementSize(aRequest) {
> +    if (this.context == "chrome") {
> +      try {
> +        let el = this.curBrowser.elementManager.getKnownElement(aRequest.element, this.getCurrentWindow());
> +        let clientRect = el.getBoundingClientRect();  

trailing whitespace

::: testing/marionette/marionette-listener.js
@@ +676,5 @@
> + */
> +function getElementSize(msg){
> +  try {
> +    let el = elementManager.getKnownElement(msg.json.element, curWindow);
> +    let clientRect = el.getBoundingClientRect();  

trailing whitespace
Attachment #673046 - Flags: review?(mdas) → review+
https://hg.mozilla.org/mozilla-central/rev/e0ddf5c89441
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in before you can comment on or make changes to this bug.