837676 - [video] Implement new startup loading events

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Video, defect, P2)

Description

[Julien Wajsberg [:julienw]]

We need to measure when the app is usable by the user. For that we'll need to send an event (the moment is specific to the app) to |window| that the performance test will be able to receive.

The event name can be "x-moz-perf-user-ready" amongst all apps so that the performance test can be similar.

For the Video app, it should be triggered when the user can actually manipulate the app.

We should measure 2 runs (cold and warm) for this app: the first run will populate the internal DB, the second run will directly load the internal DB and should be faster.

Comment 1

[:Eli Perelman]

Bug 996038 introduces new events outlining the phases of application startup. Each of these 5 events needs to be implemented.

Comment 2

[Hema Koka [:hema]]

Adding to sprint4 for discussion during planning

Comment 3

[:Eli Perelman]

As an FYI, this implementation needs to land in 2.0 as it is important for meeting release performance acceptance criteria.

https://wiki.mozilla.org/FirefoxOS/Performance/Release_Acceptance

Comment 4

[Hema Koka [:hema]]

Russ, please take this one. You can take a look at this one as an example impl. https://bugzilla.mozilla.org/show_bug.cgi?id=837674

Thanks
Hema

Comment 5

[Russ Nicoletti [:russn]]

Attachment: Pull request: https://github.com/mozilla-b2g/gaia/pull/21134

Comment 6

[John Hu [:johnhu][:johu][:醬糊小弟]]

Comment on attachment 8447538 [details] [review]
Pull request: https://github.com/mozilla-b2g/gaia/pull/21134

It looks good.

Only one concern that we already have a variable called firstScanEnd which is set before all UI update. And we introduce another variable firstScanDone at this patch. Maybe, we can use local variable to keep the state and dispatch the event for it.

Comment 7

[Russ Nicoletti [:russn]]

Thanks, John. I updated the PR to use firstScanEnded instead of introducing a new variable.

Master: https://github.com/mozilla-b2g/gaia/commit/7e3aa22cf947f30512e6124d32b594e151876640

Comment 8

[:Eli Perelman]

Russ, could you please have this uplifted to 2.0? See comment 3.

Comment 9

[Russ Nicoletti [:russn]]

v2.0:

https://github.com/mozilla-b2g/gaia/commit/78278822d534072df149f6a20cca1e8f8f49744a