Closed Bug 936310 Opened 6 years ago Closed 6 years ago

[Window Management] If lockscreen is disabled would get Javascript Error

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: alive, Assigned: jlal)

References

Details

Attachments

(1 file)

STR:
1. Disable lockscreen

Actual:
Stop at init logo.

Race condition that homescreen is not ready and we fire showwindow(unlock) event too quickly.
JamesLal this seems doesn't harm on real device. Only error log.
Summary: [Window Management] If lockscreen is disabled we cannot boot → [Window Management] If lockscreen is disabled would get Javascript Error
I didn't try this on device but it was breaking my workflow using b2g-desktop because it could not be unlocked (until recently :() by touch events.
Comment on attachment 829802 [details] [review]
mozilla-b2g:master PR#13547

quick hack fix + tests
Attachment #829802 - Flags: review?(alive)
Comment on attachment 829802 [details] [review]
mozilla-b2g:master PR#13547

\O/
Attachment #829802 - Flags: review?(alive) → review+
https://github.com/mozilla-b2g/gaia/commit/9682496ad319313fdf6a8a30bd0ccd88a2ab4537
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
I recalled that you showed me there's a "app" directory and we could put fake app there but I couldn't find where's it now.
James?
Assignee: nobody → jlal
Flags: needinfo?(jlal)
I used apps in my original patch (the one where I found this bug) here is an example of how to use this: https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/media_playback_test.js#L8-L19
Flags: needinfo?(jlal)
You need to log in before you can comment on or make changes to this bug.