Closed
Bug 1050493
Opened 10 years ago
Closed 10 years ago
Moving element may leave traces with HWA off
Categories
(Core :: Layout, defect)
Tracking
()
VERIFIED
FIXED
mozilla35
Tracking | Status | |
---|---|---|
firefox35 | --- | verified |
People
(Reporter: obrufau, Assigned: mstange)
References
Details
(Keywords: regression)
Attachments
(4 files, 3 obsolete files)
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.
Attachment #8469554 -
Attachment is obsolete: true
Attachment #8469557 -
Attachment is obsolete: true
Attachment #8469559 -
Attachment is obsolete: true
Comment 5•10 years ago
|
||
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
Component: General → Layout
Flags: needinfo?(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.
Flags: needinfo?(obrufau)
Comment 7•10 years ago
|
||
Thank you. Confirmed with HWA off, 34.0a1 (2014-08-07), Win 7 x64.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: Moving element may leave traces → Moving element may leave traces with HWA off
This demo directly sets top and left properties from a given list of values, without jQuery.
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•10 years ago
|
||
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.
Flags: needinfo?(mstange)
Keywords: regression
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Comment 11•10 years ago
|
||
Attachment #8493890 -
Flags: review?(roc)
Attachment #8493890 -
Flags: review?(roc) → review+
Assignee | ||
Comment 12•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/50eddcdf4cbb
Comment 13•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/50eddcdf4cbb
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Reporter | ||
Comment 14•10 years ago
|
||
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);
Assignee | ||
Comment 15•10 years ago
|
||
You're right, I noticed that after I checked it in, but since it's just test code it doesn't matter.
Comment 16•10 years ago
|
||
Verified fixed 35.0a1 (2014-09-25) Win 7
Status: RESOLVED → VERIFIED
status-firefox35:
--- → verified
You need to log in
before you can comment on or make changes to this bug.
Description
•