Kartikaya Gupta (email:kats@mozilla.staktrace.com) Reporter
	Description • 10 years ago

Bug 928833 enabled it on B2G. I want to let it bake for a day or two and then enable it on all platforms.

	Timothy Nikkel (:tnikkel) Assignee
	Comment 1 • 9 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6ef5e2be41b7

	Timothy Nikkel (:tnikkel) Assignee
	Comment 2 • 9 years ago

The bc1 failure in there seems to have been caused by something else, subsequent try pushes don't have it.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com) Reporter
	Comment 3 • 9 years ago

Note that event regions is hitched to APZ now so that if APZ is enabled, event regions gets auto-enabled even if the pref is false. However there are still some correctness issue with event regions (see dep bugs, although some of those are purely optimization) that are not covered by automated testing but will still need to be fixed.

	Florian Bender
	Comment 4 • 9 years ago

(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #3)
> Note that event regions is hitched to APZ now so that if APZ is enabled,
> event regions gets auto-enabled even if the pref is false. 

So Event Regions cannot be enabled without APZ? Then the pref should go.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com) Reporter
	Comment 5 • 9 years ago

Event regions can be enabled without APZ.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com) Reporter
	Comment 6 • 8 years ago

Since APZ is enabled everywhere now (on Nightly at least), this bug should be closed. I'm going to move the remaining open dependencies over to block bug 1157834 instead. I'll make that bug more general to cover any event-regions issues, whether they are perf or correctness.