Open Bug 1150941 Opened 6 years ago Updated 5 months ago

Make the tests from bug 1148855 pass (at least those that are disabled due to layer activity timeout or event regions)

Categories

(Core :: Layout, defect)

x86
macOS
defect
Not set
normal

Tracking

()

Tracking Status
firefox40 --- affected

People

(Reporter: mstange, Unassigned)

References

Details

Attachments

(1 file)

Bug 1148855 added a few tests that fail with event regions, and of those a few passed intermittently (so I disabled them completely). We should fix things so that they always pass.
This is the changeset that disabled them: https://hg.mozilla.org/integration/mozilla-inbound/rev/c61a8d4b2963
Keywords: leave-open
Once APZ is on by default everywhere tests that are skip-if(asyncPanZoom) will be no-ops, so we should either get those tests re-enabled or get rid of them. Marking this as blocking bug 1178298 so that it shows up in the APZ bug tree.
Based on the try push at https://treeherder.mozilla.org/#/jobs?repo=try&revision=62583d014b6f I think this one test at least is safe to re-enable.
Attachment #8637659 - Flags: review?(mstange)
Comment on attachment 8637659 [details] [diff] [review]
Re-enable pull-background-5

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

Looks like I never debugged why the test failed after I had tried to re-enable it in bug 1148515, so if it's passing now, that's great.
Attachment #8637659 - Flags: review?(mstange) → review+
Markus, should this bug be closed?
Flags: needinfo?(mstange)
The important tests have been enabled, but the "animated-position" ones are still disabled, for layer activity timeout reasons. Let's leave this bug open for those. I'll remove this bug from the all-aboard-apz blocking list.

We'll probably need bug 1151346 if we want to have reliable tests with animated left/top/right/bottom properties.
No longer blocks: all-aboard-apz
Flags: needinfo?(mstange)
The leave-open keyword is there and there is no activity for 6 months.
:svoisen, maybe it's time to close this bug?
Flags: needinfo?(svoisen)

The leave-open keyword is there and there is no activity for 6 months.
:svoisen, maybe it's time to close this bug?

Flags: needinfo?(svoisen)

The leave-open keyword is there and there is no activity for 6 months.
:svoisen, maybe it's time to close this bug?

Flags: needinfo?(svoisen)
Flags: needinfo?(svoisen)

The leave-open keyword is there and there is no activity for 6 months.
:svoisen, maybe it's time to close this bug?

Flags: needinfo?(svoisen)
Flags: needinfo?(svoisen)
Keywords: leave-open
You need to log in before you can comment on or make changes to this bug.