Closed Bug 1068467 Opened 10 years ago Closed 10 years ago

[Action Menu] Action menu not showing up after call screen close

Categories

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

x86
macOS
defect
Not set
normal

Tracking

(blocking-b2g:2.0M+, b2g-v2.0 affected, b2g-v2.0M verified, b2g-v2.1 verified, b2g-v2.2 verified)

RESOLVED FIXED
2.1 S7 (24Oct)
blocking-b2g 2.0M+
Tracking Status
b2g-v2.0 --- affected
b2g-v2.0M --- verified
b2g-v2.1 --- verified
b2g-v2.2 --- verified

People

(Reporter: chens, Assigned: chens)

References

Details

Attachments

(4 files, 1 obsolete file)

STR:
1. Invoke an activity with more than one choices
2. Action menu shows up for users to choose
3. Having an incoming call, wait for call screen shows up
4. Accept/Decline the call, call screen close
5. Invoke another activity with more than one choices again

Actual result:
After call screen close, action menu will close and not able to invoke any activity with more than one choices

Expect result: 
After call screen close, action menu will resume and able to invoke any activity with more than one choices
Attached file Pull request (obsolete) —
Attachment #8490542 - Flags: review?(alive)
Attached file Pull request to master
Or we can dismiss action menu when attentionopen, so when call ends it will go back to previous step before activity shows.
Attachment #8490546 - Flags: review?(alive)
Comment on attachment 8490546 [details] [review]
Pull request to master

Pref this one.

I wonder why hide does not call oncancel but anyway.
Attachment #8490546 - Flags: review?(alive) → review+
Reopen and qawanted for branch check, we may hit this problem on 2.0 and 2.1 branch.
Status: RESOLVED → REOPENED
Keywords: qawanted
Resolution: FIXED → ---
Update tracking flag based on bug 1067705 comment 11
Attached file Pull request to 2.0
NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):
[User impact] if declined: Unable to call any activity after accept a phone call.
[Testing completed]: Manual test + unit test
[Risk to taking this patch] (and alternatives if risky): Trivial one-liner change and no alternatives.
[String changes made]: None

This patch already lands on 2.0m, and maybe we need this on 2.0.
Carry review from alive in bug 1067705.
Attachment #8500823 - Flags: review+
Attachment #8500823 - Flags: approval-gaia-v2.0?
Comment on attachment 8490546 [details] [review]
Pull request to master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):
[User impact] if declined: Unable to call any activity after accept a phone call.
[Testing completed]: Manual test + unit test
[Risk to taking this patch] (and alternatives if risky): Trivial one-liner change and no alternatives.
[String changes made]: None
Attachment #8490546 - Attachment description: Pull request - 2 → Pull request to master
Attachment #8490546 - Flags: approval-gaia-v2.1?
Attachment #8490542 - Attachment is obsolete: true
This is generic bug. Set blocking-b2g to 2.0+
Blocks: Woodduck
blocking-b2g: --- → 2.0+
(In reply to Sherman Chen [:chens] from comment #5)
> Reopen and qawanted for branch check, we may hit this problem on 2.0 and 2.1
> branch.

Why did you reopen this bug? Who's doing the branch checks?
This is 2.0 blocker and we haven't fixed it yet.
[Blocking Requested - why for this release]:

(In reply to Josh Cheng from comment #9)
> This is generic bug. Set blocking-b2g to 2.0+

Why are we blocking 2.0 on this. Please note we need to confirm if this a stop-ship regression for partner at this point to land anything on 2.0. I don't see any QA/partner comments asking that here, hence I am questioning the blocking decision.
blocking-b2g: 2.0+ → 2.0?
Flags: needinfo?(jocheng)
Comment on attachment 8490546 [details] [review]
Pull request to master

Given the trivial change and since we have a few days for FC, approving this uplift on 2.1
Attachment #8490546 - Flags: approval-gaia-v2.1? → approval-gaia-v2.1+
Land on 2.0M first.
blocking-b2g: 2.0? → 2.0M+
Flags: needinfo?(jocheng)
Comment on attachment 8490546 [details] [review]
Pull request to master

This patch needs uplift to 2.1
Status: REOPENED → RESOLVED
Closed: 10 years ago10 years ago
Resolution: --- → FIXED
Comment on attachment 8500823 [details] [review]
Pull request to 2.0

Minusing the approval per https://bugzilla.mozilla.org/show_bug.cgi?id=1068467#c12, please NI me if there is still discussion needed here to reconsider it.
Attachment #8500823 - Flags: approval-gaia-v2.0? → approval-gaia-v2.0-
We have verified this issue on Woodduck 2.0, Flame 2.1 and 2.2. But after you end call, the action menu can't resume on Woodduck 2.0 and Flame 2.1, on Flame 2.2 will resume after end call. And all device can invoke any activity with more than one choices after end call. We have uploaded new video and logcat to Bugzilla.
Found time:10:21
See attachment:1021.mp4 and logcat_1021.txt
Fail rate:3/3

Step:
1.Launch Settings.
2.Tap "Date & Time" -> "Date".
3.Use another device call the device.
4.Decline the call.

Actual result:
After call screen close, action menu will close.

Expect result: 
After call screen close, action menu will resume.

Woodduck version:
Gaia-Rev        cc690f8016b672475dc186bc7fd58aef45e684b7
Gecko-Rev       03d3ab62d5b07b915434f2d1d68495ad5915ecd2
Build-ID        20141118184148
Version         32.0
Device-Name     jrdhz72_w_ff
FW-Release      4.4.2
FW-Incremental  1416306679
FW-Date         Tue Nov 18 18:31:44 CST 2014

Flame 2.1 version:
Gaia-Rev        1b231b87aad384842dfc79614b2a9ca68a4b4ff3
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-2g34_v2_1/rev/95fbd7635152
Build-ID        20141119001205
Version         34.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141119.035246
FW-Date         Wed Nov 19 03:52:56 EST 2014
Bootloader      L1TC00011880

Flame 2.2 version:
Gaia-Rev        e64428c5b2dce5db90b75a5055077a04f4bd4819
Gecko-Rev       https://hg.mozilla.org/mozilla-central/rev/aa72ddfe9f93
Build-ID        20141119160202
Version         36.0a1
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141119.192317
FW-Date         Wed Nov 19 19:23:27 EST 2014
Bootloader      L1TC00011880
Flags: needinfo?(jocheng)
Attached video 1021.MP4
Hi Chens, Can you check Marigold QA verification result per comment 18? Thanks!
Flags: needinfo?(jocheng) → needinfo?(shchen)
This is expected behavior on 2.0 and 2.1, action menu will be dismissed and not resuming after attention window closed. We've discussed this with reviewer (see comment 1, comment 2, and comment 3) and QA verification result is the same as we wanted.
Flags: needinfo?(shchen)
You need to log in before you can comment on or make changes to this bug.