1125023 - setNFCFocus(false) is called when ShrinkingUI pops up

Status: RESOLVED DUPLICATE of bug 1124346

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

Description

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

STR:
1. Enable NFC
2. Launch Gallery app, and select a photo
3. Tap another NFC device, then Shrinking-UI pops up

At this moment I observed that setNFCFocus(false) is called on the Browser Element of the the Gallery app, which should be wrong since Gallery app is visible to user and user is going to share the phote by NFC.

Comment 1

[(inactive after 6/18) Alive Kuo [:alive]]

Fixed by bug 1124346; reopen if still happens

Comment 3

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

[REOPENED for v2.2]
Hi Alive,

Bug 1124346 is fixed on master only and it's "status-b2g-v2.2: 	unaffected" which means we won't have the patch in 2.2.
I think we can either uplift bug 1124346 to 2.2 or have a separate patch for 2.2 on this bug.
Please kindly advise which approach is better.

Comment 4

[(inactive after 6/18) Alive Kuo [:alive]]

We should uplift this and 1117633 at the same time.

Comment 5

[(inactive after 6/18) Alive Kuo [:alive]]

(In reply to Alive Kuo [:alive][NEEDINFO!] from comment #4)
> We should uplift this and 1117633 at the same time.

Correct: 1124346 and 1117633

Comment 6

[howie [:howie]]

Triage: mark as blocker since it blocks feature 2.2 bugs

Comment 7

[Tim Guan-tin Chien [:timdream] (please needinfo)]

FYR the feature-b2g: 2.2 originated from bug 1082440 comment 17.

Comment 8

[howie [:howie]]

qawanted to check if the issue is fixed in 2.2 by bug 1124346 and bug 1117633

Comment 9

[Brogan Zumwalt [Inactive]]

Is more information available about how this issue should be tested? Is "setNFCFocus(false)" something we should be looking for in a logcat, or is there a description of a visual representation we can look for?

Comment 10

[(inactive after 6/18) Alive Kuo [:alive]]

Please test v2.2/master and close this bug if it's fixed.

Comment 11

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

works in master and v2.2

Comment 12

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

However I found the regression that setFocusApp(false) won't be called when the app is in background.

Filed a follow-up bug Bug 1129841.