1045919 - JavaScript Error: "TypeError: AppWindowManager.getActiveApp(...) is null" {file: "app://system.gaiamobile.org/js/activity_window_manager.js"

Status: RESOLVED FIXED

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

Description

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

+++ This bug was initially created as a clone of Bug #1037780 +++

Boot up the phone and go to `adb logcat` - after booting up, swipe left to go into the Camera app. On or around the time the permissions dialog comes up - I chose not to share but also not remember this setting, the following JS error comes up:

E/GeckoConsole(  289): [JavaScript Error: "TypeError: AppWindowManager.getActiveApp(...) is null" {file: "app://system.gaiamobile.org/js/activity_window_manager.js" line: 107}]
I/GeckoDump(  289): XXX FIXME : Got a mozContentEvent: activity-choice

Comment 1

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

$ ./check_versions.sh
Gaia      295967a0b824a355ae9d57fb08f3632ed2ad18dd
Gecko     https://hg.mozilla.org/mozilla-central/rev/a4dcfbebcb58
BuildID   20140728040209
Version   34.0a1
ro.build.version.incremental=110
ro.build.date=Fri Jun 27 15:57:58 CST 2014
B1TC00011230

Comment 2

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Alive, any idea what's happening here?

Comment 3

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

Sounds a bug. Could you take this one if you have no other blocker?

Comment 4

[(Inactive after June) George Duan [:gduan] [:喬智]]

Attachment: PR to master

I think the root cause is, when boot up device, activeApp has not been updated to homescreen, so, when getActiveApp() will give us null.

Comment 5

[(Inactive after June) George Duan [:gduan] [:喬智]]

Comment on attachment 8464529 [details] [review]
PR to master

I forgot to set review flag.

Hi Alive,
could you help to check my solution here? Please let me know your idea.
Thanks.

Comment 6

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

Comment on attachment 8464529 [details] [review]
PR to master

After thinking, I tend to use alternative to fix all the scenario:
AppWindowManager should always returns a appWindow reference.
If getActive() is null then let's return homescreenLauncher.getHomescreen().

Comment 7

[(Inactive after June) George Duan [:gduan] [:喬智]]

Comment on attachment 8464529 [details] [review]
PR to master

Hi Alive,
could you review it? Thanks.

Comment 8

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

Comment on attachment 8464529 [details] [review]
PR to master

you NEED a test.

Comment 9

[(Inactive after June) George Duan [:gduan] [:喬智]]

Thanks,
master: https://github.com/mozilla-b2g/gaia/commit/bb825aa33095371484780ed1d0a5da417a798b57