[Window Mgmt] A bad transition occurs when going back to settings using the utility tray

NEW
Unassigned

Status

()

P3
normal
3 years ago
2 years ago

People

(Reporter: KTucker, Unassigned)

Tracking

({regression})

Other Branch
ARM
Gonk (Firefox OS)
regression
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(blocking-b2g:2.2?, b2g-v2.1 unaffected, b2g-v2.2 affected, b2g-master unaffected)

Details

(Whiteboard: [2.5-Daily-Testing] [Spark][B2G][gfx-noted], URL)

Attachments

(1 attachment)

The user will notice a bad transition when they go back to settings using the "Settings" icon in the utility tray. 

Repro Steps:
1) Update a Aries to 20150629211839
2) Open "Settings".
3) Tap the home button.
4) Open the dialer.
5) Pull down the "Utility Tray" and tap the settings icon to go back to settings.
6) Repeat until encountering the issue.

Actual:
A bad transition occurs when going back to settings using the "Settings" icon in the utility tray. 

Expected:
The transition to the settings app is smooth. 

Notes:

Environmental Variables:
Device: Aries 2.5 
Build ID: 20150629211839
Gaia: 27fe0f4261e3685187769411f2f74cff19287b19
Gecko: c26dbd63604d
Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd
Version: 42.0a1 (2.5)
Firmware Version: D5803_23.1.A.1.28_NCB.ftf
User Agent: Mozilla/5.0 (Mobile; rv:42.0) Gecko/42.0 Firefox/42.0

Repro frequency: 8/10 80%
See attached: video clip, logcat
I do not know if this occurs on the Flame 2.5

I am blocked by bug 1162535. Only one app stays open at a time. 

Device: Flame 2.5 (Full Flash)(KK)(319mb)
BuildID: 20150629010206
Gaia: 8a1e4ae522c121c5cacd39b20a5386ec9055db82
Gecko: eaf4f9b45117
Gonk: a4f6f31d1fe213ac935ca8ede7d05e47324101a4
Version: 41.0a1 (2.5) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:41.0) Gecko/41.0 Firefox/41.0
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-master: --- → affected
No longer depends on: 1178510
Flags: needinfo?(pbylenga)
Keywords: regression
Whiteboard: [2.5-Daily-Testing] [Spark] → [2.5-Daily-Testing] [Spark] [systemsfe]
Created attachment 8627448 [details]
Settingsbadtransitionlogcat.txt
Adding qawanted to see if we can reproduce on flame, perhaps upping memory will avoid bug 1162535 and allow us to reproduce this issue.
Blocks: 1171119
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(pbylenga)
Keywords: qawanted
The issue occurs on Flame with 1024MB memory. Flame 2.5 and 2.2 are affected. 2.5 exhibits the issue the same as on Aries, but 2.2 is slightly different. On 2.2 it doesn't show the issue where two windows sliding in to view from different directions, but it will show the homescreen wallpaper when transitioning.

Device: Flame 2.5
BuildID: 20150702010204
Gaia: b901c8b7be2119f4df42781aac1401ed12765460
Gecko: f5e3bacfb60e
Gonk: a4f6f31d1fe213ac935ca8ede7d05e47324101a4
Version: 42.0a1 (2.5) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:42.0) Gecko/42.0 Firefox/42.0

Device: Flame 2.2
BuildID: 20150702002503
Gaia: bd386f346eb1591fddbc84bf034b22700e7e2a58
Gecko: f16c1125b9d6
Gonk: bd9cb3af2a0354577a6903917bc826489050b40d
Version: 37.0 (2.2) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0

----

This issue does NOT occur on Flame 2.1. Transitions are seamless without showing unnecessary screens.

Device: Flame 2.1
BuildID: 20150702001208
Gaia: 7080a7c28b0242f81d689d2339dfa1177e23f48f
Gecko: c201e76c63d1
Gonk: bd9cb3af2a0354577a6903917bc826489050b40d
Version: 34.0 (2.1) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
status-b2g-v2.1: --- → unaffected
status-b2g-v2.2: --- → affected
Flags: needinfo?(ktucker)
Keywords: qawanted → regression
(Reporter)

Updated

3 years ago
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Keywords: regressionwindow-wanted
[Blocking Requested - why for this release]:

Nominating this 2.5 because this looks really bad on Aries. Let's find the regression window.
blocking-b2g: --- → 2.5?
blocking-b2g: 2.5? → 2.5+
QA Whiteboard: [QAnalyst-Triage+]
QA Contact: pcheng
Working build behavior: v2.1 behavior described in comment 4.
Broken build behavior: v2.2 behavior described in comment 4. It is less severe comparing to current behavior in master.

b2g-inbound regression window:

Last Working
Device: Flame
BuildID: 20150317170958
Gaia: 647c9c649965d5a14f32f7d09f4eacc2a9f2136e
Gecko: 805dc220f255
Version: 39.0a1 (2.5 Master)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0

First Broken
Device: Flame
BuildID: 20150317172043
Gaia: 82976655ee26e4d593d35ba900632623fb656858
Gecko: ff7779bd54da
Version: 39.0a1 (2.5 Master)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0

Last Working Gaia First Broken Gecko - no repro
Gaia: 647c9c649965d5a14f32f7d09f4eacc2a9f2136e
Gecko: ff7779bd54da

Last Working Gecko First Broken Gaia - repro
Gaia: 82976655ee26e4d593d35ba900632623fb656858
Gecko: 805dc220f255

Gaia pushlog:
https://github.com/mozilla-b2g/gaia/compare/647c9c649965d5a14f32f7d09f4eacc2a9f2136e...82976655ee26e4d593d35ba900632623fb656858

Possibly caused by changes made in Bug 1141500.
Blocks: 1141500
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(jmercado)
Keywords: regressionwindow-wanted
Chirs this seems to have been caused by the changes for Bug 1141500.  Can you take a look please?
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmercado) → needinfo?(chrislord.net)

Comment 8

3 years ago
I don't really have the time to look into this right now and the fact that it's intermittent suggests to me that it's a platform issue.

Moving my needinfo to mhenretty, as last I heard the utility tray was being modified to use native scrolling anyway, which would nullify most of the code added in bug 1141500.
Flags: needinfo?(chrislord.net) → needinfo?(mhenretty)
(In reply to Chris Lord [:cwiiis] from comment #8)
> I don't really have the time to look into this right now and the fact that
> it's intermittent suggests to me that it's a platform issue.
> 
> Moving my needinfo to mhenretty, as last I heard the utility tray was being
> modified to use native scrolling anyway, which would nullify most of the
> code added in bug 1141500.

That's bug 1178162, but it's not a 2.5 blocker like this one. Someone will have to look into this eventually, whether through landing native scrolling, or fixing the platform issue exposed here.
Flags: needinfo?(mhenretty)
Can't find it, but I remember fixing a *really* similar bug.
Not the same STR but the transition was **** out in the exact same way.

Can we see if this is still reproducible on master?
Flags: needinfo?(ktucker)
Keywords: qawanted
(Reporter)

Updated

3 years ago
Flags: needinfo?(ktucker) → needinfo?(jmercado)
This issue DOES occur in the latest Aries master build.  A poor transition is clearly visible when the user goes from the utility tray to the Settings app. I did not test the issue on a Flame device because of Comment 1 (bug 1162535). 

NOTE: I used the STR in Comment 0, however, I transitioned from the Music app to the Settings app (instead of Phone app). 

Repro Rate: 8/10 

Device: Aries Master
Build ID: 20150824130451
Gaia: d7fb5717d3e0153ac64af2c0d5c11079846d81c3
Gecko: 205427533c567b7b67bf2df3937e851f16204668
Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd
Version: 43.0a1 (Master)
Firmware Version: D5803_23.1.A.1.28_NCB.ftf
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Keywords: qawanted
Etienne see comment 11 please.  This issue still reproduces.
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmercado) → needinfo?(etienne)
Priority: -- → P3
I took another shot at this one but I really can't find anything wrong on the gaia side.

It's pretty easy to reproduce though, so I'd love for somebody from platform to look at it.
On that note, toggling [1] might help (and it's definitely not hiding the issue :))

[1] https://github.com/mozilla-b2g/gaia/blob/7d915992b2e1bde8cb64906a2b83280588dc923d/apps/system/js/app_window_manager.js#L117
Flags: needinfo?(etienne)
Moving to GFX so it gets triaged there, feel free to send it back if there's anything we can do on the gaia side!
Component: Gaia::System::Window Mgmt → Graphics
Product: Firefox OS → Core
Whiteboard: [2.5-Daily-Testing] [Spark] [systemsfe] → [2.5-Daily-Testing] [Spark]

Comment 15

3 years ago
Hi Vincent, can you please check if you can help.
Flags: needinfo?(vliu)

Comment 16

3 years ago
(In reply to Duane Dixon [:ddixon] from comment #11)
> This issue DOES occur in the latest Aries master build.  A poor transition
> is clearly visible when the user goes from the utility tray to the Settings
> app. I did not test the issue on a Flame device because of Comment 1 (bug
> 1162535). 
> 
> NOTE: I used the STR in Comment 0, however, I transitioned from the Music
> app to the Settings app (instead of Phone app). 
> 
> Repro Rate: 8/10 
> 
> Device: Aries Master
> Build ID: 20150824130451
> Gaia: d7fb5717d3e0153ac64af2c0d5c11079846d81c3
> Gecko: 205427533c567b7b67bf2df3937e851f16204668
> Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd
> Version: 43.0a1 (Master)
> Firmware Version: D5803_23.1.A.1.28_NCB.ftf
> User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0

What code base of aries you used to look into this issue? KK or L?
Flags: needinfo?(ddixon)

Comment 17

3 years ago
It's KK.
Flags: needinfo?(ddixon)

Comment 18

3 years ago
I tried to sync with today's aries-kk build, and the git info for Gaia/Gecko are as follows

Gaia:  commit f75bd584aca0a751a5bed115800250faa8412927  (Good case)
       Date:   Mon Oct 19 12:09:58 2015 +0530
Gecko: commit d10e5964b98cc7d4b7a6a827f3fcddc0f0e97e84
       Date:   Sun Oct 18 11:19:30 2015 -0700

With this code base, I can't reproduce this issue. But if I rollback gaia commit to just Comment 11 said, I can see this issue easily.

Gaia: d7fb5717d3e0153ac64af2c0d5c11079846d81c3  (Bad case)

From behavior, it seems that transition from from Music to Settings is not running smoothly. Also, it seems that some further bug after August 24 fixing this issue. It would be better git bisecting gaia between good and bad case to narrow down the issue.
I'm not able to reproduce this on master either. While the platform side should be looked at, we shouldn't block on this if it's no longer reproducible, right?
Flags: needinfo?(mhenretty)
qawanted to get another retest on the latest master.
Flags: needinfo?(mhenretty)
Keywords: qawanted
Confirmed that this issue is no longer occurring on central. But it is still occurring on 2.2 as described at comment 4 where during transitioning homescreen wallpaper will show.

Not occurring on:
Device: Aries 2.5
BuildID: 20151020162500
Gaia: 3491a49da4d4ce45b8a682e95d0ce68892c2b8ea
Gecko: f7b746b4e91307448cb0746a41f677bfc23908b0
Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd
Version: 44.0a1 (2.5) 
Firmware Version: D5803_23.1.A.1.28_NCB.ftf
User Agent: Mozilla/5.0 (Mobile; rv:44.0) Gecko/44.0 Firefox/44.0

Device: Flame 2.5
BuildID: 20151020031318
Gaia: 3491a49da4d4ce45b8a682e95d0ce68892c2b8ea
Gecko: f7b746b4e91307448cb0746a41f677bfc23908b0
Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd
Version: 44.0a1 (2.5) 
Firmware Version: v18Dv4
User Agent: Mozilla/5.0 (Mobile; rv:44.0) Gecko/44.0 Firefox/44.0

Bug occurs on:
Device: Flame 2.2
BuildID: 20151020032711
Gaia: 885647d92208fb67574ced44004ab2f29d23cb45
Gecko: 6b4e563acaf9
Gonk: bd9cb3af2a0354577a6903917bc826489050b40d
Version: 37.0 (2.2) 
Firmware Version: v18Dv4
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
status-b2g-master: affected → unaffected
Flags: needinfo?(jmercado)
Keywords: qawanted
Michael this issue no longer occurs on 2.5 so do we still need the blocking tag?  Are there any plans to do more fixes for 2.2? cause we might be able to just close this completely if not.
Flags: needinfo?(jmercado) → needinfo?(mhenretty)

Comment 23

3 years ago
Thanks.
Nominate this bug as 2.2?. Let's see if partner wants it to be fixed in 2.2.
blocking-b2g: 2.5+ → 2.2?
Flags: needinfo?(vliu)
Thanks Jayme and Ken.
Flags: needinfo?(mhenretty)
Whiteboard: [2.5-Daily-Testing] [Spark] → [2.5-Daily-Testing] [Spark][B2G]
Version: unspecified → Other Branch
Whiteboard: [2.5-Daily-Testing] [Spark][B2G] → [2.5-Daily-Testing] [Spark][B2G][gfx-noted]
You need to log in before you can comment on or make changes to this bug.