1922709 - `dom.isInView` may change the DOM with setting `Element.style.pointerEvents`

Status: RESOLVED FIXED

(Remote Protocol :: Agent, defect, P3)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

The method sets el.style.pointerEvents to "auto", then, finally restores it to the original value. However, this may cause empty style attribute (which was not there before) staying. That will cause the new test for bug 1921171 which check outerHTML of editing host failing.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1922709 - Make dom.isInView restore original style attribute

It sets el.style.pointerEvents to "auto". Then, it causes updating (or
adding) style attribute of el. Finally, it'll restore the original
el.style.pointerEvents value, but unnecessary style attribute may stay
on el. Therefore, it should restore original style attribute` state too.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ded9bea493bf
Make dom.isInView restore original style attribute r=webdriver-reviewers,jgraham

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/ded9bea493bf