951321 - [B2G][Messages] Create New Message button gets selected but does not open New Message page

Status: RESOLVED FIXED

(Core :: Panning and Zooming, defect)

Description

[Nikolai Khristoforov]

Attachment: Logcat

On the empty Messages app landing page, long-pressing on the screen with two fingers for at least ten seconds results in you not being able to enter the New Message page. You can select the New Message button and it will show up as being selected, but the New Message page does not open. This issue occurs only when APZ is enabled in Settings.

Repro Steps:
1) Updated Buri to BuildID: 20131216140918
2) In Settings > Device Information > More Information > Developer, turn on "Enable APZ for all content," "Remote debugging," and "Gaia debug traces."
3) Press the Home button and open the Messages app.
4) Long-press on the screen with two fingers for at least 10 seconds and select the New Message button.
5) Verify the button gets selected but the New Message page does not open.

Actual:
The New Message button is selected but the New Message page does not open.

Expected:
The New Message page opens.

Environmental Variables:
Device: Buri 1.3 MOZ
BuildID: 20131216140918
Gaia: 504cf9a988cd84760b4040706ccc41e508a97fd2
Gecko: 55b75ce2263e7d2a8cbfdabc7e5206d9caa89f98
Version: 28.0a2
Firmware Version: V1.2_20131115

Repro frequency: 8/10
See attached: Logcat and Screenshot

Comment 1

[Nikolai Khristoforov]

Attachment: Screenshot

Comment 2

[Rick Waldron [:rwaldron]]

(In reply to Nikolai Khristoforov from comment #0)
> Repro Steps:
> 1) Updated Buri to BuildID: 20131216140918

I cannot reproduce on Inari and my Buri has never been dev-capable.

> 2) In Settings > Device Information > More Information > Developer, turn on
> "Enable APZ for all content," "Remote debugging," and "Gaia debug traces."
> 3) Press the Home button and open the Messages app.
> 4) Long-press on the screen with two fingers for at least 10 seconds and
> select the New Message button.
> 5) Verify the button gets selected but the New Message page does not open.
> 
> Actual:
> The New Message button is selected but the New Message page does not open.
> 
> Expected:
> The New Message page opens.
> 
> Environmental Variables:
> Device: Buri 1.3 MOZ
> BuildID: 20131216140918
> Gaia: 504cf9a988cd84760b4040706ccc41e508a97fd2

I can't find this commit in Gaia (see: https://github.com/mozilla-b2g/gaia/commit/504cf9a988cd84760b4040706ccc41e508a97fd2)

Comment 3

[Nikolai Khristoforov]

This bug was discovered during APZ Testing on a special build developed by Naoki.

Additional Notes: 
This issue only occurs when the "Enable APZ for all content" option is selected on the Developer menu. If the option is not selected, then the New Message page opens. 

Also, a similar issue occurs in the Contacts app. If you long-press with two fingers on the Contacts list, the Contacts setting page and New Contact page do not open even though the button looks selected.

Comment 4

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Hi Rick.  You have to look in the aurora/1.3 repo : https://github.com/mozilla-b2g/gaia/tree/v1.3

Comment 5

[Julien Wajsberg [:julienw]]

Rick: to provide a little bit of context, we want to enable APZC on 1.3, and therefore there is a lot of QA going on with this option enabled. That does sometimes mean we'll need to fix bugs by landing workarounds in SMS, and sometimes that there is a Gecko fix.

The first step is to understand why this bug is happening under APZC, and what's happening exactly.

Comment 6

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

That's a very fun step to reproduce :)

Once the app is in this state the only way to get it working is to restart it.

Comment 7

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Vivien Nicolas (:vingtetun) (:21) from comment #6)
> That's a very fun step to reproduce :)
> 
> Once the app is in this state the only way to get it working is to restart
> it.

I was wrong. It works too if you hit the home button and then go relaunch the app.

Comment 8

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

I wonder if the current state in AsyncPanZoomController is not broken somehow.

Comment 9

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Apparently you don't need to stay long. A very quick touch with 2 fingers broke the app for me.

Comment 10

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: bug951321

Seems like we are stuck in a state in GestureEventListener.cpp where we are waiting for pinch gesture. This bug happens only if you don't start a pinch. The issue also dissapears as soon as you start to pinch.

Comment 12

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

https://hg.mozilla.org/integration/b2g-inbound/rev/a9182a077284

Comment 13

[Ryan VanderMeulen [:RyanVM]]

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

Target milestone corresponds to when it lands on m-c. Status flags track branch uplifts.

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/f89e855ad21f