Seth Miko [:SMiko] Reporter
	Description • 10 years ago

Description: While a file is downloading, partially opening the notification center causes it to stick to the screen. Hitting the home button will take the user back to home, but the focus will remain on the partially opened notification center. 

Repro Steps:
1: Update a Flame to 20141010000201
2: Download a large file
3: While the file is downloading, open Music or Dialer and pull the notification tray down 1/4 of the way.

Actual: The notification center sticks partially open and retains focus

Expected: The notification center does not stick partially open. 

Flame 2.1 KK (319mb) (Full Flash)

Environmental Variables:
Device: Flame 2.1
BuildID: 20141010000201
Gaia: bc8eb493311c58f1f311a56b8b645b52bfbd2f71
Gecko: 72c13d8631ff
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 34.0a2 (2.1)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Repro frequency: 100%

See attached: logcat

Video clip:http://youtu.be/hoT-y-BBwH8

	Seth Miko [:SMiko] Reporter
	Comment 1 • 10 years ago

This issue DOES repro on Flame 2.2 (319mb/full flash/20141009040206) and Flame 2.2 (319mb/full flash/20141010040202)

Actual result: The notification center sticks partially open and retains focus.

Device: Flame 2.2
BuildID: 20141009040206
Gaia: 7b92615bdc97e5c675cd385ec68bc5e47e0c5288
Gecko: f0bb13ef0ee4
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 35.0a1 (2.2)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0

Device: Flame 2.2
BuildID: 20141010040202
Gaia: 1036b544b7e102592bd9fab95cd9317329ac1293
Gecko: 50b689feab5f
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 35.0a1 (2.2)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0

	Peter Bylenga [:PBylenga]
	Comment 2 • 10 years ago

QAWanted for branch checks.

	Jayme Mercado [:JMercado] (Inactive 3/4/2016)
	Comment 3 • 10 years ago

This issue does NOT occur on 2.0 Flame KK.

Environmental Variables:
Device: Flame 2.0
BuildID: 20141010074705
Gaia: 6effca669c5baaf6cd7a63c91b71a02c6bd953b3
Gecko: 54ec9cb26b59
Version: 32.0 (2.0) 
Firmware Version: L1TC10011800
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

	Joshua Mitchell (Inactive)
	Comment 4 • 10 years ago

Triage - Not nomming - after discussing this bug it seems difficult to encounter in a normal user-usage

	Seth Miko [:SMiko] Reporter
	Comment 5 • 10 years ago

After further investigation, we have nailed down a consistent repro using these STR:

1: Update a Flame to 20141010000201
2: Open at least 4 apps to slow performance.
3: Perform edge gesture to switch between the opened apps.
4: While the app is loading (app icon shown with white background), pull the notification tray down halfway. 

Actual: The notification tray gets stuck.

Expected: The notification tray fully opens. 

Video clip: http://youtu.be/8Kw4xFkb92w

	Joshua Mitchell (Inactive)
	Comment 6 • 10 years ago

Updating title to match new STR

NI to Notification Menu (windows management) owner for a blocking call - with the updated steps it seems like this might be easier to encounter now. It looks bad but is easy to recover from

	Gerry Chang [:gchang]
	Comment 7 • 10 years ago

[Blocking Requested - why for this release]:

Hi Howie, 
Per comment #6, It's much easier to repro the issue. This might need your input to determine if it's 2.1 blocker.

	Gregor Wagner [:gwagner]
	Comment 8 • 10 years ago

Bad UX issue.

	Guillaume Marty [:gmarty] Assignee
	Comment 9 • 10 years ago

This bug happens because a touchEnd event is not triggered. While we investigate it on the platform side, I worked on this patch that calls the onTouchEnd() method when no new touchMove events are recorded within a second.

Also, although https://bugzilla.mozilla.org/show_bug.cgi?id=1081365#c1 above says v2.2 is affected, I wasn't able to repro and the video is the same than the bug on v2.1.
For this reason the patch is based on v2.1.

Seth, would you mind to double check that this bug affects v2.2? If so, can you please provide a solid STR?

Etienne, can you have a look at this PR?

	Etienne Segonzac (:etienne)
	Comment 10 • 10 years ago

Comment on attachment 8517394 [details] [review]
Github PR

Just want to double-check before doing the review: did you try listening for "touchcancel"?

	Guillaume Marty [:gmarty] Assignee
	Comment 11 • 10 years ago

Comment on attachment 8517394 [details] [review]
Github PR

Yes but unfortunately the touchcancel event is never fired either :-(

	Seth Miko [:SMiko] Reporter
	Comment 12 • 10 years ago

I am unable to repro on Flame 2.2 (319mb/full flash) or Flame 2.2 (319mb/shallow flash) using the STR in comment #5.

Actual result: The notification tray does not get stuck. 

Device: Flame 2.2
BuildID: 20141105040206
Gaia: 7c9e7cabbde941b976e0e40a3a1d94e21aa9c5e9
Gecko: 62990ec7ad78
Gonk: 48835395daa6a49b281db62c50805bd6ca24077e
Version: 36.0a1 (2.2)
Firmware: V188
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

	Joshua Mitchell (Inactive)
	Comment 13 • 10 years ago

changing 2.2 tracking flag to unaffected

	Etienne Segonzac (:etienne)
	Comment 14 • 10 years ago

Comment on attachment 8517394 [details] [review]
Github PR

This was driving me crazy but I think I tracked it down to fullscreen app opening while the tray is being dragged.

Guillaume, will send you a patch for review soon :)

	Etienne Segonzac (:etienne)
	Comment 15 • 10 years ago

What do you think?

	Etienne Segonzac (:etienne)
	Comment 16 • 10 years ago

Comment on attachment 8518193 [details] [review]
Gaia PR

Talked with Guillaume, this is not fixing the issue completely.

We went back and forth on this a bit, and this is yet again a case where we're going to do an ugly workaround that wouldn't be needed if the utility tray part of bug 1061969 got uplifted.

Guillaume will make a (small) PR accordingly with the utility tray part of bug 1061969 and the quick follow up from bug 1082650.

	Guillaume Marty [:gmarty] Assignee
	Comment 17 • 10 years ago

Here is the PR I made for Bug 1084060 that uplifts the utility tray code from Bug 1061969.
It's rebased and fixes the bug described here.

	Etienne Segonzac (:etienne)
	Comment 18 • 10 years ago

Comment on attachment 8518908 [details] [review]
Github PR

I think we're missing the part from
https://github.com/gmarty/gaia/commit/668b3d30ebd64f755fc1d0726be18f51190daf93

And we should remove the "setActive" part for this 2.1 version (don't think we have the gecko support and we want to make this as short and simple as possible :))

	Guillaume Marty [:gmarty] Assignee
	Comment 19 • 10 years ago

Comment on attachment 8518908 [details] [review]
Github PR

I reworked the PR according to your suggestions. How does it look now?

	Etienne Segonzac (:etienne)
	Comment 20 • 10 years ago

Comment on attachment 8518908 [details] [review]
Github PR

looking good, looking good.
(needs rebasing but nothing too bad)

	Guillaume Marty [:gmarty] Assignee
	Comment 21 • 10 years ago

Comment on attachment 8518908 [details] [review]
Github PR

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): Utility tray visual refresh.
[User impact] if declined: The utility tray get sometime stuck in the middle, not completely opened, nor closed when CPU is under pressure. Also this patch improves the performance of swiping up/down the tray and feels much smoother.
[Testing completed]: Unit testing coverage is good and manual testing is required.
[Risk to taking this patch] (and alternatives if risky): Though this patch looks big, it is basically an uplift from master and have been there for months now and has proven to be completely stable.
[String changes made]: None.

	Guillaume Marty [:gmarty] Assignee
	Comment 22 • 10 years ago

Marking as resolved so that it appears on Fabrice's radar.

	Michael Henretty [:mikehenrty][:mhenretty]
	Comment 23 • 10 years ago

Guillaume, the try run is looking pretty bad. Might be some harness issues, or there might be real failures in there. In any case, can you figure out what is going on before we land this?

https://treeherder.mozilla.org/ui/#/jobs?repo=gaia-try&revision=db5510d24c35

	Ryan VanderMeulen [:RyanVM]
	Comment 24 • 10 years ago

Looks like a combination of infra issues and maybe-real problems. Can we get a fresh Gaia Try run for it?

	Michael Henretty [:mikehenrty][:mhenretty]
	Comment 25 • 10 years ago

Guillaume, we fixed a bunch of issues with the tests on v2.1 in bug 1103962, so try rebasing your patch and see if it's cleared up.

	Guillaume Marty [:gmarty] Assignee
	Comment 26 • 10 years ago

Thanks Michael. I rebased and the JS marionette tests are passing now. I fixed some issues in python but some tests are timing out now. I will take a look with the help of :zac tomorrow and hopefully land it.

	Guillaume Marty [:gmarty] Assignee
	Comment 27 • 10 years ago

Tests are green! Thanks :zac
Landed in v2.1 in https://github.com/mozilla-b2g/gaia/commit/a60bebd38a743c97635a49fd62159266ee434155

	Yeojin Chung [:YeojinC]
	Comment 28 • 10 years ago

This issue is verified fixed on Flame 2.1.

Result: Following the STRs in Comment 5, the notification tray fully opens and does not get stuck when the user pulls it down halfway.

Device: Flame 2.1 (319mb, KK, Full Flash)
BuildID: 20141201001201
Gaia: ccb49abe412c978a4045f0c75abff534372716c4
Gecko: 18fb67530b22
Gonk: 48835395daa6a49b281db62c50805bd6ca24077e
Version: 34.0 (2.1)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0