1004121 - bookmark cold launch regression 210ms Apr28

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia, defect, P1)

Description

[Hubert Figuiere [:hub]]

https://datazilla.mozilla.org/b2g/?branch=master&device=hamachi&range=7&test=startup_time&app_list=bookmark&app=bookmark&gaia_rev=49a096509f5ea53d&gecko_rev=4d926af89907&plot=median

We have a 210ms regression in cold launch in the "bookmark" application on hamachi

Before
Gaia ddcc24e4f37de64faf2387267de48db8a59956c8
Gecko ea25086073c8
1099ms


After
Gaia cba203c750d3364f3e3971ea8a0ec315ced784d3
Gecko 4d926af89907
1316ms

Comment 1

[Ben Francis [:benfrancis]]

Hmm, we don't have a component for the bookmark application yet, but it's nothing to do with the browser app.

Christian, do you have any ideas about this?

Comment 2

[Hubert Figuiere [:hub]]

1. I couldn't reproduce the regression here on hamachi

2. this doesn't seem to be a real application but just an activity. So maybe the testing is just irrelevant.

Comment 3

[Cristian Rodriguez (:crdlc)]

I don't have idea how/where it is calculated... Basically if it calculates the time until the info is displayed on the form, then I can say that, when the activity is launched, the activity goes to the datastore in order to know if the user is adding a new one or updating a bookmark which exists on the device because we have to show different UIs depending on the state.

https://github.com/mozilla-b2g/gaia/blob/master/apps/bookmark/js/bookmark_editor.js#L12

Comment 4

[Hubert Figuiere [:hub]]

Marking as WONTFIX. This this is an activity, two things are gonna happen. Bug 1005917 will disable testing this in |make test-perf|. And bug 1007421 is to test the bookmarks.