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

+++ This bug was initially created as a clone of Bug #1272757 +++

Back when the APZ drag code was first added in bug 1199885, drag blocks were only created if the apz.drag.enabled pref was enabled, and they were always created as unconfirmed [1][2]. The idea was that if they hit a scrollbar, they would be confirmed by the main-thread sending over drag metrics [3], and thus would get processed.

However, in bug 1242690 I made it so that drag blocks get created on mousedown, even if apz.drag.enabled is not set. However, I overlooked this business with every drag block being unconfirmed by default. This means that if apz.drag.enabled is false, every drag block ends up waiting for the full content timeout before it proceeds.

I fixed part of this issue in bug 1272757, for the cases where the drag block lands on a confirmed hit area. For the dispatch-to-content hit areas, we still need to make the base widget send a APZ target confirmation, somewhere in [4]. This bug is tracking that issue.

[1] http://mxr.mozilla.org/mozilla-central/source/gfx/layers/apz/src/APZCTreeManager.cpp?rev=09c5d774bd83#701
[2] http://mxr.mozilla.org/mozilla-central/source/gfx/layers/apz/src/InputQueue.cpp?rev=09c5d774bd83#185
[3] http://mxr.mozilla.org/mozilla-central/source/gfx/layers/apz/src/InputQueue.cpp?rev=09c5d774bd83#643
[4] http://mxr.mozilla.org/mozilla-central/source/widget/nsBaseWidget.cpp?rev=114ca1fc9c51#1121

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

https://treeherder.mozilla.org/#/jobs?repo=try&revision=38c6abfb6eca

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

Hopefully better: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b734e996752b

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

Although the tests are all passing on this one, I see that test_group_touchevents is taking ~40s longer on android and that's a sign something is getting stuck. All other platforms seem ok.

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

Review commit: https://reviewboard.mozilla.org/r/58246/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/58246/

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

Review commit: https://reviewboard.mozilla.org/r/58248/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/58248/

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

This patch also cleans up some inconsistencies in the conditions under which the
main thread would respond to wheel and mouse events. With this patch applied, the
main-thread notifications are only sent if the input block id is nonzero, which
indicates the APZ actually processed the input event and added it to an input
block.

Review commit: https://reviewboard.mozilla.org/r/58250/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/58250/

	Randall Barker [:rbarker]
	Comment 7 • 8 years ago

Comment on attachment 8760790 [details]
Bug 1273137 - Add a mochitest that creates a drag block on a dispatch-to-content-region.

https://reviewboard.mozilla.org/r/58246/#review55088

	Randall Barker [:rbarker]
	Comment 8 • 8 years ago

Comment on attachment 8760791 [details]
Bug 1273137 - Add the mHandledByAPZ flag to MouseInput as well.

https://reviewboard.mozilla.org/r/58248/#review55090

	Randall Barker [:rbarker]
	Comment 9 • 8 years ago

Comment on attachment 8760792 [details]
Bug 1273137 - Add the missing main-thread target confirmation for drag blocks.

https://reviewboard.mozilla.org/r/58250/#review55092

	Pulsebot
	Comment 10 • 8 years ago

Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1cd13955b729
Add a mochitest that creates a drag block on a dispatch-to-content-region. r=rbarker
https://hg.mozilla.org/integration/mozilla-inbound/rev/2bc047b037a0
Add the mHandledByAPZ flag to MouseInput as well. r=rbarker
https://hg.mozilla.org/integration/mozilla-inbound/rev/b72ab182dd55
Add the missing main-thread target confirmation for drag blocks. r=rbarker

	Carsten Book [:Tomcat]
	Comment 11 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/1cd13955b729
https://hg.mozilla.org/mozilla-central/rev/2bc047b037a0
https://hg.mozilla.org/mozilla-central/rev/b72ab182dd55

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

This can ride the trains.