1050493 - Moving element may leave traces with HWA off

Status: VERIFIED FIXED

(Core :: Layout, defect)

Description

[obrufau]

Attachment: Screenshot

User Agent: Mozilla/5.0 (Windows NT 5.1; rv:34.0) Gecko/20100101 Firefox/34.0 (Beta/Release)
Build ID: 20140807030202

Steps to reproduce:

Open the attached demo, and let the element move.


Actual results:

At some points it will leave traces.


Expected results:

It shouldn't.

Comment 1

[obrufau]

Attachment: Demo

Comment 2

[obrufau]

Attachment: Screenshot 2

Comment 3

[obrufau]

Attachment: Demo

Comment 4

[obrufau]

Attachment: Demo

Comment 5

[Paul Silaghi, QA [:pauly]]

Can't reproduce, 34.0a1 (2014-08-07), Win 7 x64.
Please check if the issue occurs using Firefox in safe mode (with your addons disabled):
http://support.mozilla.com/kb/Safe+Mode

And on a new, empty profile:
http://support.mozilla.org/en-US/kb/Managing-profiles#w_starting-the-profile-manager

Comment 6

[obrufau]

(In reply to Paul Silaghi, QA [:pauly] from comment #5)
> Can't reproduce, 34.0a1 (2014-08-07), Win 7 x64.
> Please check if the issue occurs using Firefox in safe mode (with your
> addons disabled):
> http://support.mozilla.com/kb/Safe+Mode
> 
> And on a new, empty profile:
> http://support.mozilla.org/en-US/kb/Managing-profiles#w_starting-the-profile-
> manager

Yes, I already tested it on a new profile.

Maybe it only happens on Win XP, or with with HWA off.

Comment 7

[Paul Silaghi, QA [:pauly]]

Thank you.
Confirmed with HWA off, 34.0a1 (2014-08-07), Win 7 x64.

Comment 8

[obrufau]

Attachment: Demo 2

This demo directly sets top and left properties from a given list of values, without jQuery.

Comment 9

[obrufau]

Good: 32.0a1 2014-05-13 (4b6d63b05a0a)
Bad: 32.0a1 2014-05-14 (2f8af55d6e9a)
Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=4b6d63b05a0a&tochange=2f8af55d6e9a

Latest versions:
 * 35.0a1 2014-09-22 - affected
 * 34.0a2 2014-09-22 - affected
 * 33.0b5 - unaffected
 * 32.0.2 - unaffected

Comment 10

[Paul Silaghi, QA [:pauly]]

Thanks obrufau!

Last good revision: 971b840c40ff
First bad revision: 35af5d1c97c3
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=971b840c40ff&tochange=35af5d1c97c3

Markus, it seems to be one of your patches.

Comment 11

[Markus Stange [:mstange]]

Attachment: patch

Comment 12

[Markus Stange [:mstange]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/50eddcdf4cbb

Comment 13

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/50eddcdf4cbb

Comment 14

[obrufau]

Comment on attachment 8493890 [details] [diff] [review]
patch

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

::: layout/reftests/bugs/1050493-1.html
@@ +36,5 @@
> +  }
> +}
> +window.addEventListener("MozReftestInvalidate", function () {
> +  f();
> +});

I think a wrapper function is not needed:
> window.addEventListener("MozReftestInvalidate", f);

Comment 15

[Markus Stange [:mstange]]

You're right, I noticed that after I checked it in, but since it's just test code it doesn't matter.

Comment 16

[Paul Silaghi, QA [:pauly]]

Verified fixed 35.0a1 (2014-09-25) Win 7