1059079 - Increase wait_for_homescreen timeout and patch URL check

Status: RESOLVED FIXED

(Testing :: Mozbase, defect)

Description

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

We're getting anywhere from 50-100 second startups of the emulator in Mnw tests, and with Bug 900551 doing a lot of IPC on startup now, this has increased even more, sometimes causing the homescreen wait to timeout.

Also, in the wait for homescreen function, we only look for "ftu" or "homescreen". However, as of FxOS 2.0 homescreen is now "verticalhome", so we should look for that when triggering. There are better ways to do this (via checking settings in the system), but they're rather complicated so I'll file a followup for that.

Comment 1

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Attachment: Patch 1 (v1) - Increase wait_for_homescreen timeout and add watch for new homescreen name

Adding multiple reviewers because I can't land Bug 900551 without this. Whoever gets to it first wins!

Comment 2

[Malini Das [:mdas] - Away, not checking bugmail]

Comment on attachment 8479599 [details] [diff] [review]
Patch 1 (v1) - Increase wait_for_homescreen timeout and add watch for new homescreen name

Review of attachment 8479599 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm, thanks

Comment 3

[Andrew Halberstadt [:ahal] (PTO until March)]

Comment on attachment 8479599 [details] [diff] [review]
Patch 1 (v1) - Increase wait_for_homescreen timeout and add watch for new homescreen name

Review of attachment 8479599 [details] [diff] [review]:
-----------------------------------------------------------------

Looks like I lost, but this wfm too. Did we want to file a bug to get this value from mozSettings? Or is it not worth it.

Comment 4

[Malini Das [:mdas] - Away, not checking bugmail]

(In reply to Andrew Halberstadt [:ahal] from comment #3)
> Comment on attachment 8479599 [details] [diff] [review]
> Patch 1 (v1) - Increase wait_for_homescreen timeout and add watch for new
> homescreen name
> 
> Review of attachment 8479599 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Looks like I lost, but this wfm too. Did we want to file a bug to get this
> value from mozSettings? Or is it not worth it.

https://bugzilla.mozilla.org/show_bug.cgi?id=1058089 is filed, I guess we can morph that bug to modify this change so we use mozSettings

Comment 5

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

https://hg.mozilla.org/integration/b2g-inbound/rev/c6f54d821c11

Comment 6

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

https://hg.mozilla.org/integration/b2g-inbound/rev/43a4f13eeb31

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/43a4f13eeb31