Closed
Bug 1005800
Opened 11 years ago
Closed 11 years ago
[NFC] Long press home button still can work while NFC is pairing
Categories
(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)
Tracking
(blocking-b2g:2.0+, b2g-v2.0 fixed)
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | fixed |
People
(Reporter: ashiue, Assigned: gasolin)
References
Details
(Whiteboard: [FT:System-Platform][p=1])
Attachments
(1 file)
+++ This bug was initially created as a clone of Bug #997595 +++
most up-to-date pvt build 2014/4/16
STR:
1. launch "settings" app for 2 phones
2. turn on "NFC" for 2 phones
3. launch "browser" from phone A
4. tap 2 phones
5. long press home button on phone A when shrinking UI shows up
Expected result:
Home button should block all functions while NFC is pairing
Actual result:
Long press home button still can kill processes, once kill browser app, tap 2 phones still cause them to pair(vibrate)
===============================================================
Verified on:
Gaia f55fc5c507312c7aac51ec9bb73061fd4ed5c5fb
Gecko https://hg.mozilla.org/mozilla-central/rev/3285e030d9c0
BuildID 20140504160202
Version 32.0a1
Long press home button still works when NFC is pairing.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → gasolin
Reporter | ||
Updated•11 years ago
|
blocking-b2g: --- → 2.0?
Assignee | ||
Comment 2•11 years ago
|
||
The issue happens because CardsView got `holdhome` event before ShrinkingUI, so the CardsView event handler is still triggered.
Which is preferred approach, to reorder event receiving(?) or check ShrinkingUI stat in CardsView?
Status: NEW → ASSIGNED
Flags: needinfo?(alive)
Comment 3•11 years ago
|
||
re-nominate. IMO, this is more like a defect of already-landed feature.
Blocks: b2g-NFC-2.0
blocking-b2g: --- → 2.0?
Comment 4•11 years ago
|
||
(In reply to Fred Lin [:gasolin] from comment #2)
> The issue happens because CardsView got `holdhome` event before ShrinkingUI,
> so the CardsView event handler is still triggered.
>
> Which is preferred approach, to reorder event receiving(?) or check
> ShrinkingUI stat in CardsView?
CardsView should not depends on ShrinkingUI. ShrinkingUI is for NFC-device only but CardsView is for every device. please reorder it.
I suggest to wait https://bugzilla.mozilla.org/show_bug.cgi?id=935260 landed.
Flags: needinfo?(alive)
Updated•11 years ago
|
blocking-b2g: 2.0? → 2.0+
Assignee | ||
Updated•11 years ago
|
Depends on: task-manager
Comment 5•11 years ago
|
||
Propose to have this fixed in sprint3, as per comment 4 we should wait for bug 935260 which is in sprint2.
Feel free to modify the target milestone if this doesn't work.
Target Milestone: --- → 2.0 S3 (6june)
Assignee | ||
Comment 7•11 years ago
|
||
No, since the blocking bug landed, I'll start work on it next week
Flags: needinfo?(gasolin)
Assignee | ||
Comment 9•11 years ago
|
||
The approach:
In order to receive `holdhome` event first, move nfc import in front of task manager import in index.html
Updated•11 years ago
|
Attachment #8428561 -
Flags: review?(alive) → review+
Assignee | ||
Comment 10•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
status-b2g-v2.0:
--- → fixed
Reporter | ||
Comment 11•11 years ago
|
||
Verified on
Gaia bc6f07c149770c6e6dfbea941ac65138dc364a15
Gecko https://hg.mozilla.org/mozilla-central/rev/e017c15325ae
BuildID 20140528040205
Version 32.0a1
Status: RESOLVED → VERIFIED
Comment 12•11 years ago
|
||
verify with Flame v180 + v2.0 gaia/gecko, it's fine
Gaia 7edd3b0b9f65c3dde235c732d270e43e055a1254
Gecko https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/f3639e825b3b
BuildID 20140915135336
Version 32.0
Keywords: verifyme
You need to log in
before you can comment on or make changes to this bug.
Description
•