Closed Bug 989286 Opened 11 years ago Closed 11 years ago

[Tarako][Keyboard][Personas] It takes more than 5 seconds for keyboard to get ready

Categories

(Firefox OS Graveyard :: Gaia::Keyboard, defect)

All
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 992346

People

(Reporter: bli, Unassigned)

References

Details

(Keywords: perf)

Environment: -------------------------- Gaia ecbd79aab76e119a7987092b59cbafc97f7ba72f Gecko f23735074dc673457426afa5d87a67f934a18a9b BuildID 20140328060053 Version 28.1 Steps to reproduce: ---------------------------- 1. Open the Marketplace app 2. Select the gear icon on the top right corner 3. Select button "Sign In / Sign Up". 4. Tap on the input box, and wait for the keyboard to get ready. Actual results: ---------------------------- 1. It takes more than 5 seconds for keyboard to get ready. 2. Sometimes the keyboard does not show up.
Blocks: 989003
blocking-b2g: --- → 1.3T?
Bingqing: Does this happen only on marketplace apps ? Or is this a keyboard regression across core apps like sms, mms, email etc ? :mlee, would be be abple to help with target keyboard launchtime metrics here in case of core apps and marketplace for the keyboard to appear and help understand if this would be a blocker.
Flags: needinfo?(mlee)
Flags: needinfo?(bli)
(In reply to bhavana bajaj [:bajaj] from comment #1) > Bingqing: Does this happen only on marketplace apps ? Or is this a keyboard > regression across core apps like sms, mms, email etc ? > No, not only on marketplace, but it is more obvious when try to login marketplace. And with other apps, it cannot be constantly reproduced.
Flags: needinfo?(bli)
ni? Rudy do you mind taking an initial look at this ? thanks
Flags: needinfo?(rlu)
Sorry that I have other blockers to look at, so may not have time to check this. Bingqing, May I know if this is a regression or not, could it be reproduced with an earlier build? Thanks.
Flags: needinfo?(rlu)
Flags: needinfo?(bli)
(In reply to Rudy Lu [:rudyl] from comment #4) > > Bingqing, > > May I know if this is a regression or not, could it be reproduced with an > earlier build? > Thanks. I don't think this is a regression. The delay happens not only when try to sign in/sign up persona, but also in other circumstances, for example, add contacts, add email accounts, edit email and so on. Tarako always has this issue.
Flags: needinfo?(bli)
batch update: 1.3T? so we can get more eyes into these marketplace related bugs
do we have Click sound, Auto correction and word suggestion turned off when observing this symptom? thanks
Flags: needinfo?(bli)
(In reply to Joe Cheng [:jcheng] from comment #7) > do we have Click sound, Auto correction and word suggestion turned off when > observing this symptom? thanks No, they are not turned off. On build 20140328060053, defaults are on. I still can reproduce this on build 20140404164001. (I did not change the default settings.) On build 20140404164001, default settings of auto correction and word suggestion are off, but click sound is on.
Flags: needinfo?(bli)
can we test again with Click sound, Auto correction and word suggestion turned off ? and only leave vibration on? Thanks
Flags: needinfo?(bli)
(In reply to Joe Cheng [:jcheng] from comment #9) > can we test again with Click sound, Auto correction and word suggestion > turned off ? > and only leave vibration on? Thanks Yes, it still can be reproduced with only vibration on.
Flags: needinfo?(bli)
ni? ehung
Flags: needinfo?(ehung)
I enabled debug log in keyboard_manager.js of System app, and see the following debug message. - Before starting testing E/GeckoConsole( 81): Content JS LOG at app://system.gaiamobile.org/js/keyboard_manager.js:91 in km_debug: [Keyboard Manager] set layout to display: type=text index=0 - When the screen shows "connecting to persona..." E/GeckoConsole( 82): Content JS LOG at app://system.gaiamobile.org/js/keyboard_manager.js:91 in km_debug: [Keyboard Manager] mozmemorypressure event E/GeckoConsole( 82): Content JS LOG at app://system.gaiamobile.org/js/keyboard_manager.js:91 in km_debug: [Keyboard Manager] mozmemorypressure event; keyboard removed - When input shows and click on it E/GeckoConsole( 82): Content JS LOG at app://system.gaiamobile.org/js/keyboard_manager.js:91 in km_debug: [Keyboard Manager] get focus event email E/GeckoConsole( 82): Content JS LOG at app://system.gaiamobile.org/js/keyboard_manager.js:91 in km_debug: [Keyboard Manager] set layout to display: type=email index=0 - After waiting for a while (> 5sec) E/GeckoConsole( 82): Content JS LOG at app://system.gaiamobile.org/js/keyboard_manager.js:91 in km_debug: [Keyboard Manager] resizeKeyboard: 205 E/GeckoConsole( 82): Content JS LOG at app://system.gaiamobile.org/js/keyboard_manager.js:91 in km_debug: [Keyboard Manager] showKeyboard display transitionend E/GeckoConsole( 82): Content JS LOG at app://system.gaiamobile.org/js/keyboard_manager.js:91 in km_debug: [Keyboard Manager] updateHeight: 205 So it can be sure that memory pressure happens and kills keyboard app *before* keyboard app is becoming foreground. The problem is launching a keyboard app takes time. :(
Flags: needinfo?(ehung)
Tim, any way to speed up keyboard launching or reduce persona memory usage?
Flags: needinfo?(timdream)
Component: Performance → Gaia::Keyboard
Keywords: perf
See Also: → 992346
The log in comment 12 confirms the keyboard management works correctly and this is indeed a dup of bug 992346. I am dup the older bug to the newer one since Rudy have some more information on the launching time there.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(timdream)
Resolution: --- → DUPLICATE
blocking-b2g: 1.3T? → ---
Flags: needinfo?(mlee)
See Also: 992346
Wil, This bug is now dup'd to bug 992346, but we need you to look at this to tell if the same steps will still take 5 sec after we changed the marketplace app to the lightweighted version. We would like to weight in your feedback to decide whether or not to 1.3T+ bug 992346. Thanks!
Flags: needinfo?(clouserw)
(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TPE) (please ni?) from comment #16) > Wil, > > This bug is now dup'd to bug 992346, but we need you to look at this to tell > if the same steps will still take 5 sec after we changed the marketplace app > to the lightweighted version. We would like to weight in your feedback to > decide whether or not to 1.3T+ bug 992346. Thanks! Note - bug 992346 technically happens in other use cases outside of persona. One other known place is when you are trying to create a message. Nicole also mentioned that she was seeing this problem quite commonly in third party app input fields as well. So it's likely a more general issue with keyboard cold launch, rather than specific to one STR.
Before comment 17 I was going to say the phone is likely very stressed with both marketplace and persona running, and the new lightweight marketplace doesn't use persona, so I'd guess it would be fine. If it's visible elsewhere though, I have less confidence about that. I think for v1 of the new app the only input field is search.
Flags: needinfo?(clouserw)
(In reply to Jason Smith [:jsmith] from comment #17) > So it's likely a more general issue with keyboard cold launch, rather than > specific to one STR. Yes, hence the dup.
You need to log in before you can comment on or make changes to this bug.