Closed Bug 1139238 Opened 6 years ago Closed 6 years ago

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

Categories

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

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)

RESOLVED FIXED
blocking-b2g 2.0M+
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)

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!
Status: NEW → RESOLVED
Closed: 6 years ago6 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.