1159618 - The built-in keyboard app would not be run as OOP when using WebIDE to debug

Status: RESOLVED DUPLICATE of bug 1160980

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

Description

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

STR
===
1. Use WebIDE to debug keyboard app
   1.a Enable debugging via USB on device, with Settings > Developer
   1.b Request higher privileges with these steps,
       https://developer.mozilla.org/en-US/docs/Tools/WebIDE/Running_and_debugging_apps#Unrestricted_app_debugging_(including_certified_apps.2C_main_process.2C_etc.)

2. After phone reboots, found that keyboard app was not run as OOP.

Comment 1

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

After some investigation, I found input window manager would get 0 as the total memory so that it won't run keyboard app as OOP (will do when mem >= 512).

Comment 2

[GitHub Autolander Bot]

Attachment: [gaia] RudyLu:system/FeatureDetector > mozilla-b2g:master

Comment 3

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

I suspect this is related to bug 1150423, so set the dependency.

Comment 4

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Comment on attachment 8599208 [details] [review]
[gaia] RudyLu:system/FeatureDetector > mozilla-b2g:master

Hi Alive, Chens,

It seems there are cases that deviceMemory was not assigned with correct value and then input window manager would get 0 from this module and then caused this issue.

Not sure this fits in the correct code pattern of system module, so ask for your opinions first.
Thanks.

Comment 5

[Sherman Chen [:chens] (inactive)]

Comment on attachment 8599208 [details] [review]
[gaia] RudyLu:system/FeatureDetector > mozilla-b2g:master

I found reboot is the key to repro this issue, and the fix looks good to me.

Comment 6

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8599208 [details] [review]
[gaia] RudyLu:system/FeatureDetector > mozilla-b2g:master

Could we do this in FeatureDetector.js and have Keyboard to call Service.request('getDeviceMemory').then(....) on start? Thx.

Comment 7

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Dup to the newer bug that has an r+'d patch.