1242396 - [TV 2.5] FxOS TV Metrics - Track "send to TV" usage

Status: RESOLVED FIXED

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

Description

[KM Lee [:rexboy] (inactivate after Sep. 22)]

We need to track "send to TV" usage.

Comment 1

[KM Lee [:rexboy] (inactivate after Sep. 22)]

[Blocking Requested - why for this release]:

Comment 2

[GitHub Autolander Bot]

Attachment: [gaia] rexboy7:1242396-sendToTvMetrics > mozilla-b2g:master

Comment 3

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Comment on attachment 8712528 [details] [review]
[gaia] rexboy7:1242396-sendToTvMetrics > mozilla-b2g:master

This bug actually just records invocation counts of notification receiver, since it's receiver of "send to TV" feature and it's launched at each request.

The cons is that we just can't tell this is metrics of send to TV if we don't document it well. What I can come up for now is that we just duplicate the invocation count to a new entry named like "sendToTVCount." but it'll still be under the entry of notification receiver.

Luke can you review it? And may I ask your opinion about the concern above. Thanks!

Comment 4

[Luke Chang [:lchang] (inactive)]

Hi Joe,

In order to track "send to TV" feature without huge modifications for now, we intend to count the launching times of the apps, "notification-receiver" and "fling-player", and regard them as the count of "send tab to TV" and "send video to TV" respectively. Do you think it's feasible?

Comment 5

[Joe Cheng [:jcheng] (please needinfo)]

(In reply to Luke Chang [:lchang] from comment #4)
> Hi Joe,
> 
> In order to track "send to TV" feature without huge modifications for now,
> we intend to count the launching times of the apps, "notification-receiver"
> and "fling-player", and regard them as the count of "send tab to TV" and
> "send video to TV" respectively. Do you think it's feasible?

it sounds ok to me if we can have the count of "send tab to TV" and "send video to TV" respectively but it's a bit too technical and outside of my expertise. also ni? SC to see if this captures meaningful data to show how presentation API is being used

Comment 6

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

sgtm!

Comment 7

[Luke Chang [:lchang] (inactive)]

Comment on attachment 8712528 [details] [review]
[gaia] rexboy7:1242396-sendToTvMetrics > mozilla-b2g:master

It's good to go with only a few nits on the GitHub. Thanks.

Comment 8

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Once we confirm installOrigin equals to marketplace, marketplace would get corrected.
So let's just close it now.

Comment 9

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Sorry, ignore comment 8.

Comment 10

[KM Lee [:rexboy] (inactivate after Sep. 22)]

https://github.com/mozilla-b2g/gaia/commit/4a3ba8682f6108321e5a4fe0adff694d343dbb27

Comment 11

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Comment on attachment 8712528 [details] [review]
[gaia] rexboy7:1242396-sendToTvMetrics > mozilla-b2g:master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): feature
[User impact] if declined: can't log "send to tv" usage
[Testing completed]: local done
[Risk to taking this patch] (and alternatives if risky): small
[String changes made]: none

Comment 12

[Josh Cheng [:josh]]

Comment on attachment 8712528 [details] [review]
[gaia] rexboy7:1242396-sendToTvMetrics > mozilla-b2g:master

Approve for TV 2.5

Comment 13

[Carsten Book [:Tomcat]]

https://github.com/mozilla-b2g/gaia/commit/0158b93758e8185befceb8bc9f7d6951f209f2ec