Closed
Bug 998954
Opened 11 years ago
Closed 11 years ago
LockScreen and parts of System app not working anymore at all
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(blocking-b2g:2.0+, b2g-v2.0 fixed)
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | fixed |
People
(Reporter: gerard-majax, Assigned: gduan)
References
Details
(Keywords: regression)
Device is all black, logcat shows a lot of errors related to l10n:
E/GeckoConsole( 76): [JavaScript Error: "TypeError: tokens is null" {file: "app://system.gaiamobile.org/shared/js/l10n_date.js" line: 32}]
Reporter | ||
Comment 1•11 years ago
|
||
E/GeckoConsole( 76): [JavaScript Error: "NS_ERROR_FILE_NOT_FOUND: " {file: "app://system.gaiamobile.org/shared/js/l10n.js" line: 219}]
I/power ( 76): *** set_screen_state 1
I/Gecko ( 76): ############################### browserElementPanning.js loaded
I/Gecko ( 76): ######################## BrowserElementChildPreload.js loaded
E/GeckoConsole( 76): [JavaScript Error: "TypeError: navigator.mozL10n.ready is not a function" {file: "app://system.gaiamobile.org/js/bootstrap.js" line: 92}]
Reporter | ||
Comment 2•11 years ago
|
||
It seems this is only getting reproduced with OTA packages :(.
I'm wondering if it may not be related to some recent build system changes on Gaia.
blocking-b2g: --- → 2.0?
Keywords: dataloss
Comment 3•11 years ago
|
||
(In reply to Alexandre LISSY :gerard-majax from comment #2)
> It seems this is only getting reproduced with OTA packages :(.
>
> I'm wondering if it may not be related to some recent build system changes
> on Gaia.
So we've confirmed this doesn't happen if you directly flash the device without doing an upgrade, right?
Reporter | ||
Comment 4•11 years ago
|
||
(In reply to Jason Smith [:jsmith] from comment #3)
> (In reply to Alexandre LISSY :gerard-majax from comment #2)
> > It seems this is only getting reproduced with OTA packages :(.
> >
> > I'm wondering if it may not be related to some recent build system changes
> > on Gaia.
>
> So we've confirmed this doesn't happen if you directly flash the device
> without doing an upgrade, right?
At least, I could not reproduce after doing an APP=system install-gaia on this device. Please note that the logcat error warns about missing l10n_date.js file, but this one is present in the application.zip.
I don't have the issue on another device updated via OTA too, but with a build I did yesterday. I'm currently testing updating this device to a build from now, and testing a local revert of commit b6d1c7fae27ec22ee06e3da7898fe2565fc0a8c0 (bug 934727) as suggested by RyanVM on IRC.
Reporter | ||
Updated•11 years ago
|
Summary: Gaia not working anymore at all → LockScreen and parts of System app not working anymore at all
Reporter | ||
Comment 5•11 years ago
|
||
Unrelated to bug 934727.
Reporter | ||
Comment 6•11 years ago
|
||
Jason, I can also reproduce after a doing a make clean a then reinstalling gaia only, via install-gaia.
Comment 7•11 years ago
|
||
Ok - let me get someone to confirm this on our side.
Natalya - Can you have someone flash a trunk build to see if we can reproduce this?
Flags: needinfo?(nkot)
Reporter | ||
Comment 8•11 years ago
|
||
I'm doing bisect on gaia, I have a good lead for now.
Flags: needinfo?(jsmith)
Reporter | ||
Comment 9•11 years ago
|
||
Confirmed. A local revert of 6b7a10e5d026cf031b2f3fc0fe5f671ea02dd4f9 makes the system okay:
> $ git revert -m 1 6b7a10e5d026cf031b2f3fc0fe5f671ea02dd4f9
> $ make clean && make MOZILLA_OFFICIAL=1 PRODUCTION=1 NOFTU=1 REMOTE_DEBUGGER=1 GAIA_DEV_PIXELS_PER_PX=1.5 LOCALE_BASEDIR=locales/ LOCALES_FILE=locales/languages_all.json APP=system install-gaia
So the regression is likely coming from bug 968661
Comment 10•11 years ago
|
||
Alright - let's get that patch backed out then.
Flags: needinfo?(nkot)
Flags: needinfo?(jsmith)
Comment 11•11 years ago
|
||
Bug 968661 reverted.
Assignee: nobody → gduan
Status: NEW → RESOLVED
Closed: 11 years ago
status-b2g-v2.0:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 2.0 S1 (9may)
Updated•11 years ago
|
blocking-b2g: 2.0? → 2.0+
Assignee | ||
Comment 12•11 years ago
|
||
Hi Alex,
with my commit, I cannot see the log you mention in comment 0 and comment 1 by
> $ make clean && make MOZILLA_OFFICIAL=1 PRODUCTION=1 NOFTU=1 REMOTE_DEBUGGER=1 GAIA_DEV_PIXELS_PER_PX=1.5 LOCALE_BASEDIR=locales/ LOCALES_FILE=locales/languages_all.json APP=system install-gaia
and also, system app works fine.
Maybe I can also have your build version?
Flags: needinfo?(lissyx+mozillians)
Reporter | ||
Updated•11 years ago
|
Flags: needinfo?(lissyx+mozillians)
Reporter | ||
Comment 13•11 years ago
|
||
(In reply to George Duan [:gduan] [:喬智] from comment #12)
> Hi Alex,
> with my commit, I cannot see the log you mention in comment 0 and comment 1
> by
> > $ make clean && make MOZILLA_OFFICIAL=1 PRODUCTION=1 NOFTU=1 REMOTE_DEBUGGER=1 GAIA_DEV_PIXELS_PER_PX=1.5 LOCALE_BASEDIR=locales/ LOCALES_FILE=locales/languages_all.json APP=system install-gaia
>
> and also, system app works fine.
> Maybe I can also have your build version?
After investigating a lot with George on IRC, we found out that the error was triggered by the change of l10n.js: my gaia clone was quite an old one, from the time where we had a apps/NAME/shared/ directory populated. This made the build system changes getting confused and pulling this into build_stage/, hence breaking the whole.
You need to log in
before you can comment on or make changes to this bug.
Description
•