Closed Bug 1139238 Opened 10 years ago Closed 10 years ago

[FFOS2.0][Woodduck][Airplane mode]The airplane mode is invalid

Categories

(Firefox OS Graveyard :: Gaia::Settings, defect, P2)

Product:
Firefox OS Graveyard
Component:
Gaia::Settings
Type:
defect

Tracking

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

Status:
RESOLVED FIXED
Project Flags:
blocking-b2g 2.0M+
Tracking Flags:
Tracking Status
b2g-v2.0 --- affected
b2g-v2.0M --- fixed
b2g-v2.1 --- affected
b2g-v2.1S --- affected
b2g-v2.2 --- affected
b2g-master --- fixed

People

(Reporter: sync-1, Assigned: GaryChen, NeedInfo)

References

Details

Attachments

(8 files)

adb log
472.24 KB, text/plain
Details
mtklog
446.67 KB, application/octet-stream
Details
[gaia] mpizza:Bug_1139238_v2.0m > mozilla-b2g:v2.0m
46 bytes, text/x-github-pull-request
Details | Review
[gaia] mpizza:Bug_1139238_master > mozilla-b2g:master
46 bytes, text/x-github-pull-request
arthurcc
: review+
Details | Review
video part2
8.00 MB, application/octet-stream
Details
video part3
3.55 MB, application/octet-stream
Details
video part1
8.00 MB, application/octet-stream
Details
[gaia] mpizza:bug_1139238_v2.0m_airplanemode_enhance > mozilla-b2g:v2.0m
46 bytes, text/x-github-pull-request
Details | Review
DEFECT DESCRIPTION: The airplane mode is invalid REPRODUCING PROCEDURES: 1.Insert SIM2 ，Activate the cell with MCC=001,send some CB(0,65534,919,921)to SIM2 2.Change the MCC to 730,send some CB(0,65534,919,921)to SIM2 3.Pull down the notification bar then activate the airplane mode 4.Redo the sstep 2,3 =>KO EXPECTED BEHAVIOUR: ASSOCIATE SPECIFICATION: TEST PLAN REFERENCE: TOOLS AND PLATFORMS USED: USER IMPACT: REPRODUCING RATE: For FT PR, Please list reference mobile's behavior:
Attached file adb log —
Attached file mtklog —
Hi Peipei, Can we reproduce this? Thanks!
Blocks: Woodduck
Flags: needinfo?(pcheng)
(In reply to Josh Cheng [:josh] from comment #3) > Hi Peipei, > Can we reproduce this? > Thanks! Josh, we cannot test this because we don't have lab environment :(
Flags: needinfo?(pcheng)
hi mozilla: 步骤：下拉状态栏，连续点击几次蓝牙开关，之后点击飞行模式图标，接着反复切换点击蓝牙和飞行模式图标，复现问题，概率8/10,附件为对应log. 该问题应该是由于快速多次切换飞行模式后，就会在settings中看到飞行模式的开关与实际情况不符或者一直处于无法设置状态。但是通知栏中还是可以点击的。
Hi Gary, Can we introduce some delay for frequently on-off bluetooth? Will this be the same as wifi? Thanks!
Flags: needinfo?(gchen)
Assignee: nobody → gchen
Flags: needinfo?(gchen)
Component: Gaia::Settings → Gaia::System
Comment on attachment 8573054 [details] [review] [gaia] mpizza:Bug_1139238_v2.0m > mozilla-b2g:v2.0m Hi Tim, I add one more condition to avoid user clicks bluetooth and airplane button rapidly. Could you help to review this patch and give me some feedback? Thanks.
Attachment #8573054 - Flags: review?(timdream)
Comment on attachment 8573054 [details] [review] [gaia] mpizza:Bug_1139238_v2.0m > mozilla-b2g:v2.0m I am not sure this is a safe approach. Also please update the commit message to document the fix, not the bug. Thanks.
Attachment #8573054 - Flags: review?(timdream) → review?(arthur.chen)
Comment on attachment 8573054 [details] [review] [gaia] mpizza:Bug_1139238_v2.0m > mozilla-b2g:v2.0m It is sane ignoring the request of toggling bluetooth when airplane mode is in the transition states. Although there are other corner cases to be handled (ex: while bluetooth is enabling/disabling, can users toggle airplane mode?), but we won't have a proper fix until API reporting the hardware status are available. I left some comments regarding the implementation in github, please check, thanks.
Attachment #8573054 - Flags: review?(arthur.chen)
Hi Norry, Could you help to check all branches? Thanks.
Flags: needinfo?(fan.luo)
Keywords: qawanted
Hi Gary, Due to there is no lab environment, we can't send CB message according to comment 0. But I have tried to repro it according to comment 5, and can't repro on woodduck 2.0M, Flame 2.0/2.1/2.1s/2.2/3.0.
Flags: needinfo?(fan.luo) → needinfo?(gchen)
Hi ChenDing, Could you help to try patch per https://bugzilla.mozilla.org/attachment.cgi?id=8573054? Thanks!
Flags: needinfo?(chen.ding)
Status: NEW → ASSIGNED
Component: Gaia::System → Gaia::Settings
Comment on attachment 8574599 [details] [review] [gaia] mpizza:Bug_1139238_master > mozilla-b2g:master Hi arthur, I've addressed you comment and added some tests. Could you help to review this patch? Thanks.
Attachment #8574599 - Flags: review?(arthur.chen)
hi mozilla: I test the patch and it don't fixed this PR.
gary, per comment 16, the patch does not fix this problem. Could you please take a look?
Flags: needinfo?(gchen)
(In reply to sync-1 from comment #16) > hi mozilla: > I test the patch and it don't fixed this PR. Could you provide the STR video recording? Thanks.
Flags: needinfo?(gchen)
Flags: needinfo?(sync-1)
Created an attachment (id=1186872) video part2
Created an attachment (id=1186872) video part2
Attached file video part2 —
Created an attachment (id=1186872) video part2
Created an attachment (id=1186874) video part3
Created an attachment (id=1186874) video part3
Attached file video part3 —
Created an attachment (id=1186874) video part3
Attached file video part1 —
Created an attachment (id=1186871) video part1
sorry, set to wrong bug. reopen it.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → NEW
Comment on attachment 8574599 [details] [review] [gaia] mpizza:Bug_1139238_master > mozilla-b2g:master Looking good to me, thanks.
Attachment #8574599 - Flags: review?(arthur.chen) → review+
Hi ChenDing, Could you help to try patch per https://bugzilla.mozilla.org/attachment.cgi?id=8575824 Thanks!
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Removing qawanted since this was already been branch checked and is now resolved fixed.
QA Whiteboard: [QAnalyst-Triage+]
Keywords: qawanted
Hi Gary, DO you need 2.0M patch review? Thanks!
Flags: needinfo?(gchen)
Hi Josh, Sorry, I forget set v2.m? flag. Thanks for your reminding.
blocking-b2g: --- → 2.0M?
Flags: needinfo?(gchen)
Hi Gary, 2.0M+ Thanks!
blocking-b2g: 2.0M? → 2.0M+
Flags: needinfo?(gchen)
hi mozilla: 有一个现象想咨询一下，飞行模式在点击时设置开启或者关闭的耗时太久，有时会长达5分钟左右，是否可以优化或者能否提供一个优化的思路？
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: