David Anderson [:dvander] - inactive, e-mail if emergency Assignee
	Description • 9 years ago

One less pref to have working APZ.

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

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

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

::: layout/base/nsLayoutUtils.cpp
@@ -7655,5 @@
>  nsLayoutUtils::WantSubAPZC()
>  {
> -  // TODO Turn this on for inprocess OMTC on all platforms
> -  bool wantSubAPZC = gfxPrefs::AsyncPanZoomEnabled() &&
> -                     gfxPrefs::APZSubframeEnabled();

This function looks different as of bug 950934; your patch needs rebasing. After rebasing you can even get rid of this function and just inline the remaining APZC check into the call sites.

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

Comment on attachment 8558753 [details] [diff] [review]
rebased

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

r+ with tests deduplicated.

::: layout/reftests/async-scrolling/reftest.list
@@ +19,3 @@
>  
>  pref(layout.async-containerless-scrolling.enabled,false) skip-if(!asyncPanZoom) == bg-fixed-1.html bg-fixed-1-ref.html
>  pref(layout.async-containerless-scrolling.enabled,false) skip-if(!asyncPanZoom) == bg-fixed-cover-1.html bg-fixed-cover-1-ref.html

Some of the tests down here are now redundant. Deduplicate please.

	David Anderson [:dvander] - inactive, e-mail if emergency Assignee
	Comment 5 • 9 years ago

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/d3b41b0ead7d

w/ nits fixed

	Carsten Book [:Tomcat]
	Comment 6 • 9 years ago

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