Closed
Bug 1041528
Opened 10 years ago
Closed 10 years ago
'Enter Sim 2 Pin' overlay minimizes and input field can't be seen.
Categories
(Firefox OS Graveyard :: Gaia::System::Input Mgmt, defect)
Tracking
(blocking-b2g:2.0+, b2g-v1.4 unaffected, b2g-v2.0 fixed, b2g-v2.1 unaffected)
RESOLVED
FIXED
blocking-b2g | 2.0+ |
Tracking | Status | |
---|---|---|
b2g-v1.4 | --- | unaffected |
b2g-v2.0 | --- | fixed |
b2g-v2.1 | --- | unaffected |
People
(Reporter: gwagner, Unassigned)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
image/png
|
Details |
This seems like a regression from the last few days.
STR on flame with 2.0 07/21/2014 OTA update:
Have SIM pin enabled fro both sims.
1st PIN dialog works as expected.
2nd sim dialog starts minimized. I can see the keyboard and the top of the dialog but the input field is hidden.
Reporter | ||
Updated•10 years ago
|
blocking-b2g: --- → 2.0?
Comment 2•10 years ago
|
||
QA Wanted for branch checks.
Component: RIL → Gaia::System
Keywords: qawanted
Comment 3•10 years ago
|
||
Comment 4•10 years ago
|
||
This issue does occur on Flame 2.0. The SIM 2 pin screen is shifted and the box cannot be seen after a restart of the device. I have included a screenshot of this effect.
Device: Flame 2.0
BuildID: 20140721082721
Gaia: b9d19011123487009c80d1200937652d58c434a0
Gecko: d69cd84b6824
Version: 32.0a2 (2.0)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
This issue does not occur on the Flame 2.1 and Flame 1.4 builds. The Sim 2 pin screen does not shift.
Environmental Variables:
Device: Flame Master
BuildID: 20140722092900
Gaia: cd50cb6ddeb430ddaf43e4fcf5774b9d262a8cf3
Gecko: e5ced39f443b
Version: 34.0a1 (Master)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
Environmental Variables:
Device: Flame 1.4
BuildID: 20140722122101
Gaia: 5db3d93d68c30c1cf24499b3cda3fd881d207ea6
Gecko: 3dde355c32a1
Version: 30.0 (1.4)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0
Buri and OpenC could not be used for branch checks as these devices do not have two SIM slots.
Comment 5•10 years ago
|
||
Jayme - you did not set the tracking flags. Please fix that and re-NI me. Also, if this occured in 2.0 but not in 1.4 then it is a regression.
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(jmitchell) → needinfo?(jmercado)
Comment 6•10 years ago
|
||
Sorry about that.
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v1.4:
--- → unaffected
status-b2g-v2.0:
--- → affected
status-b2g-v2.1:
--- → affected
Flags: needinfo?(jmercado) → needinfo?(jmitchell)
Keywords: regression
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmitchell)
Keywords: regressionwindow-wanted
Reporter | ||
Updated•10 years ago
|
blocking-b2g: 2.0? → 2.0+
Reporter | ||
Comment 7•10 years ago
|
||
I guess Arthur knows this code best.
Assignee: nobody → arthur.chen
Updated•10 years ago
|
QA Contact: jmercado
Comment 8•10 years ago
|
||
Aurora Regression Window:
Last working
Environmental Variables:
Device: Flame 2.0
BuildID: 20140718084131
Gaia: dc8e1899f934e648b8de19bbba072ccadd94f8c6
Gecko: 7a18dba0208f
Version: 32.0a2 (2.0)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
First Broken
Environmental Variables:
Device: Flame 2.0
BuildID: 20140718103616
Gaia: 6ccd97a605e7f4ccad2fa92ac9575b35028e07a7
Gecko: c0f00b1268d6
Version: 32.0a2 (2.0)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
Last working gaia / First broken gecko - Does not occur
Gaia: bd2303de0be9bad0f5a3a1339c5a48c808c90e3a
Gecko: c0f00b1268d6
First broken gaia / Last working gekko - Issue DOES occur
Gaia: 6ccd97a605e7f4ccad2fa92ac9575b35028e07a7
Gecko: f2b8840032c5
Gaia Pushlog: https://github.com/mozilla-b2g/gaia/compare/bd2303de0be9bad0f5a3a1339c5a48c808c90e3a...6ccd97a605e7f4ccad2fa92ac9575b35028e07a7
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(jmitchell)
Keywords: regressionwindow-wanted
Comment 9•10 years ago
|
||
Ryan - could this have been broken by bug 980217 ?
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmitchell) → needinfo?(ryanvm)
Comment 10•10 years ago
|
||
Sounds like a question for Wei Gao.
Flags: needinfo?(ryanvm) → needinfo?(wei.gao)
Comment 11•10 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #10)
> Sounds like a question for Wei Gao.
I am sorry to hear that,
I have no phone with firefoxOS master/v2.0, so I can't test it locally. But v1.3t and v1.4 can work normal.
Maybe that patch doesn't suit master/v2.0 as system_dialog has been restructured.
So could I revert that commit on master/v2.0?
I am so sorry again for so much trouble. I will be more careful afterwards. So sorry.
Flags: needinfo?(ryanvm)
Comment 12•10 years ago
|
||
Why don't we let Alive or Julien weigh in first. If a backout is the verdict, both are in timezones that will be able to back this out earlier than I can anyway.
Flags: needinfo?(wei.gao)
Flags: needinfo?(ryanvm)
Comment 13•10 years ago
|
||
I'm really not the right person here, I merely helped Wei landing the patch :) I have needinfo alive on the regressing bug, he'll be able to help for sure.
Comment 14•10 years ago
|
||
This seems related to keyboard window. Alive, any ideas?
Flags: needinfo?(arthur.chen) → needinfo?(alive)
Reporter | ||
Comment 16•10 years ago
|
||
Moving component based on comment 14
Component: Gaia::System → Gaia::Keyboard
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][lead-review+]
Comment 17•10 years ago
|
||
Please backout if that really occurs.
Evelyn, could we have somebody from device team to help this dear contributor to fix it?
Flags: needinfo?(alive) → needinfo?(ehung)
Comment 18•10 years ago
|
||
(In reply to Alive Kuo [:alive][NEEDINFO!] from comment #17)
> Please backout if that really occurs.
> Evelyn, could we have somebody from device team to help this dear
> contributor to fix it?
I think we can just follow contributing way - if the patch Wei Gao provided causes the problem here, please back out it. Thanks.
Flags: needinfo?(ehung)
Comment 19•10 years ago
|
||
(In reply to Evelyn Hung [:evelyn] from comment #18)
> (In reply to Alive Kuo [:alive][NEEDINFO!] from comment #17)
> > Please backout if that really occurs.
> > Evelyn, could we have somebody from device team to help this dear
> > contributor to fix it?
>
> I think we can just follow contributing way - if the patch Wei Gao provided
> causes the problem here, please back out it. Thanks.
Dear Evelyn
Firstly, sorry for bringing trouble.
As this issue is caused by me, I want to try my best to fix this issue.
So, do you mean I should recommit a PR to revert that commit? If so, I am glad to do it right now.
Do I understand you correctly or some other things I missed.
Thanks.
Flags: needinfo?(ehung)
Updated•10 years ago
|
Assignee: arthur.chen → nobody
Updated•10 years ago
|
Component: Gaia::Keyboard → Gaia::System
Reporter | ||
Comment 20•10 years ago
|
||
Tim, can you help to get this resolved?
Flags: needinfo?(timdream)
Comment 21•10 years ago
|
||
Let's fix this bug by backing out v2.0 branch patch of bug 980217, and clone a follow-up bug on 2.0 branch for bug 980217.
I will talk to Alive offline so everyone have a shared understanding on how to resolve issue like this swiftly.
Blocks: 980217
Component: Gaia::System → Gaia::System::Input Mgmt
Flags: needinfo?(timdream)
Flags: needinfo?(ehung)
Comment 22•10 years ago
|
||
BTW 2.1 is unaffected based on comment 4.
Comment 23•10 years ago
|
||
Fixed by backout bug 980217 on 2.0 branch.
Reporter | ||
Updated•10 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•