1129081 - [Window Management][Power Menu] Power Menu list items do not highlight when selected.

Status: RESOLVED DUPLICATE of bug 1127066

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

Description

[Martin Shuman [:Marty]]

Attachment: logcat-Power-Menu.txt

Description:
When selecting a list item from the Power Menu, there is no animation or highlight to indicate that the item has been selected.

Repro Steps:
1) Update a Flame to 20150203055641
2) Long press the Power Button to bring up the Power Menu
3) Select 'Silence incoming calls'

Actual:
The Power Menu is dismissed, but there is no highlight to indicate the list item selected.

Expected:
The list item is highlighted, indicating it has been selected.

Environmental Variables:
Device: Flame 3.0 (Full Flash)(319MB)
Build ID: 20150203055641
Gaia: ae5a1580da948c3b9f93528146b007fc4f6a712b
Gecko: ae5d04409cd9
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 38.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:38.0) Gecko/38.0 Firefox/38.0


Repro frequency: 8/8
See attached: video clip, logcat

-------------------------------

This issue does NOT occur on Flame 2.2.
The list item is highlighted, indicating it has been selected.

Environmental Variables:
Device: Flame 2.2 (Full Flash)(319MB)
Build ID: 20150203002504
Gaia: cd62ff9fe199fb43920ba27bd5fdbc5c311016fc
Gecko: 11d93135c678
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 37.0a2 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0

Comment 1

[Peter Bylenga [:PBylenga]]

[Blocking Requested - why for this release]:
Noticeable regression of a core feature.  Missing highlight to confirm selection.

Requesting a window.

Comment 2

[Yeojin Chung [:YeojinC]]

Mozilla-inbound Regression Window:

Last Working Environmental Variables:
Device: Flame 3.0
BuildID: 20150201170935
Gaia: 740c7c2330d08eb9298597e0455f53d4619bbc1a
Gecko: 231a8c61b49f
Version: 38.0a1 (3.0) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:38.0) Gecko/38.0 Firefox/38.0

First Broken Environmental Variables:
Device: Flame 3.0
BuildID: 20150201174135
Gaia: 740c7c2330d08eb9298597e0455f53d4619bbc1a
Gecko: bcefc7d8d885
Version: 38.0a1 (3.0) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:38.0) Gecko/38.0 Firefox/38.0

Last Working Gaia First Broken Gecko: Issue DOES reproduce 
Gaia: 740c7c2330d08eb9298597e0455f53d4619bbc1a
Gecko: bcefc7d8d885

First Broken Gaia Last Working Gecko: Issue does NOT reproduce
Gaia: 740c7c2330d08eb9298597e0455f53d4619bbc1a
Gecko: 231a8c61b49f

http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=231a8c61b49f&tochange=bcefc7d8d885

Caused by bug 950934

Comment 3

[KTucker [:KTucker][Inactive 3/4/2016]]

Botond, can you take a look at this please? This might have been caused by the work done on bug 950934.

Comment 4

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

Presumably this will be fixed by hooking up the ActiveElementManager in the root process as part of bug 1127066.

Comment 5

[Botond Ballo [:botond]]

(In reply to Kartikaya Gupta (email:[e-mail]) from comment #4)
> Presumably this will be fixed by hooking up the ActiveElementManager in the
> root process as part of bug 1127066.

I would expect so. Let's retest once bug 1127066 is fixed.

Comment 6

[howie [:howie]]

Triage: regression blocking

Comment 7

[Botond Ballo [:botond]]

(In reply to Botond Ballo [:botond] from comment #5)
> (In reply to Kartikaya Gupta (email:[e-mail]) from comment #4)
> > Presumably this will be fixed by hooking up the ActiveElementManager in the
> > root process as part of bug 1127066.
> 
> I would expect so. Let's retest once bug 1127066 is fixed.

Yup, the patches I just posted to bug 1127066 fix this.

Comment 8

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

Confirmed fixed on a local master build that includes patches from bug 1127066.