1024753 - [Camera] Add Camera app to whitelistedUnifiedApps

Status: RESOLVED FIXED

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

Description

[Justin D'Arcangelo [:justindarc]]

The new startup performance events were added to Camera in Bug 1015385. However, an entry needs to be added for the Camera app to the `whitelistedUnifiedApps` array in `tests/performance/startup_events_test.js`.

Comment 1

[Justin D'Arcangelo [:justindarc]]

Attachment: pull-request (master)

Eli: Flagging you for an easy review since this is your territory :-)

Comment 2

[Justin D'Arcangelo [:justindarc]]

Landed on master:

https://github.com/mozilla-b2g/gaia/commit/ea748a0b69f22fdce7317d45ce2a7d03f5d3999e

Comment 3

[:Eli Perelman]

Comment on attachment 8439548 [details] [review]
pull-request (master)

Requesting uplift to 2.0 as it is important for meeting our release performance acceptance criteria. The associated patch for this implementation has already landed with the Camera app, so Camera is outputting the data but we cannot capture it without this entry.

[Feature/regressing bug #]: bug 996038
[User impact if declined]: none
[Describe test coverage new/current, TBPL]: Feature only triggers events for testing, no user-facing features or tests
[Risks and why]: Low, as there are no user-perceived changes
[String/UUID change made/needed]: n/a

Comment 4

[Mike Lee [:mlee]]

Bhavana + Preeti,

Can we get approval on this? It's blocking our ability to do fxOS 2.0 Performance Signoff.

Comment 5

[:Eli Perelman]

Clearing ni? based on previous approval.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

v2.0: https://github.com/mozilla-b2g/gaia/commit/597975839c04e0198eb98c2c77474f057b5531e7