1249943 - Make test_basic_pan work on more platforms

Status: RESOLVED FIXED

(Core :: Panning and Zooming, defect)

Description

[Kartikaya Gupta (email:[e-mail])]

test_basic_pan.html is an APZ mochitest that tests touch-based panning. It only runs on B2G which means it no longer runs in automation. We can make it run in Fennec and possibly Linux without too much trouble, I think. I have patches for bug 1249915 that allow synthesizing native touch events on both of those platforms. I think for Fennec we need to make sure the APZ:TransformEnd notification gets broadcast properly; right now it is only sent in the child process which Fennec doesn't have. We should probably move it to be a observer notification instead.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

Attachment: Patch

Comment 2

[Botond Ballo [:botond]]

Comment on attachment 8722499 [details] [diff] [review]
Patch

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

::: gfx/layers/apz/test/mochitest/test_basic_pan.html
@@ +45,5 @@
>          // events get held up. Try to prevent that by making long-taps require
>          // a 10 second hold.
>          ["ui.click_hold_context_menus.delay", "10000"],
> +        // The subtests in this test do touch-drags to pan the page, but we don't
> +        // want those pans to turn into fling animations, so we increase the

Why wasn't this an issue on existing platforms where the test ran?

Comment 3

[Kartikaya Gupta (email:[e-mail])]

(In reply to Botond Ballo [:botond] from comment #2)
> Why wasn't this an issue on existing platforms where the test ran?

The fling physics constants might have come into play. The fling that I was seeing was very small, and only on Linux/Windows. In mobile.js (Fennec) that pref is already a little higher than in all.js and so on Fennec I never saw the fling happen. Presumably on B2G one of the other friction constants was preventing the fling, but I didn't actually check.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/027c6cc12078

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/611823f79eb548fd9d14c7c73483c030561c98f5
Back out changeset 027c6cc12078 (bug 1249943) for making test_basic_pan.html permafail (timeout) on Linux, on a CLOSED TREE.

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Backed out for making test_basic_pan.html permafail on Linux.  (I didn't look closely at other platforms; there may have been other failures as well.)

I hope it was safe that I didn't also back out bug 1249748.

Comment 7

[Kartikaya Gupta (email:[e-mail])]

Thanks dbaron! I forgot to disable the test for non-APZ configurations, so it was failing in Linux non-e10s mochitest only. I'll update and reland.

Comment 8

[Kartikaya Gupta (email:[e-mail])]

I put the same APZ-enabled and window.TouchEvent guards into the test that I had in test_tap.html

Comment 9

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f1595343603d

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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