Closed Bug 1233823 Opened 9 years ago Closed 9 years ago

[JSMarionette] Make client.apps.launch wait for system fullyLoaded instead of homescreen

Categories

(Testing Graveyard :: JSMarionette, defect)

Product:
Testing Graveyard
Component:
JSMarionette
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: mikehenrty, Assigned: mikehenrty)

References

Details

(Whiteboard: [MJS][systemsfe])

Attachments

(1 file)

[gaia] mikehenrty:bug-1233823-wait-for-system-not-homescreen > mozilla-b2g:master
46 bytes, text/x-github-pull-request
aus
: review+
Details | Review 
We are seeing a lot of timeouts in the waitFor homescreen logic, and really we should be waiting for the system rather than the homescreen. Let's fix this.
Comment on attachment 8700149 [details] [review]
[gaia] mikehenrty:bug-1233823-wait-for-system-not-homescreen > mozilla-b2g:master

It looks like this is already helping the situation. Gareth, can you take a look?
Attachment #8700149 - Flags: review?(gaye)
Comment on attachment 8700149 [details] [review]
[gaia] mikehenrty:bug-1233823-wait-for-system-not-homescreen > mozilla-b2g:master

Or Aus.
Attachment #8700149 - Flags: review?(aus)
Comment on attachment 8700149 [details] [review]
[gaia] mikehenrty:bug-1233823-wait-for-system-not-homescreen > mozilla-b2g:master

\o/ , so much better!
Attachment #8700149 - Flags: review?(gaye)
Attachment #8700149 - Flags: review?(aus)
Attachment #8700149 - Flags: review+
commit (master): https://github.com/mozilla-b2g/gaia/commit/8b2af648a23b241f440bd85292b11be66242b1f0

fixed.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: