Refactor suggestions UI metrics recording |
||||
Issue descriptionHere are the different ways we record events in the context of ntp suggestions: - UMA recorded directly from Java: NewTabPageUMA, maybe other things - UMA recorded inderectily through the content suggestions metrics backend: SnippetsBridge - Events notified to the scheduler: SnippetsBridge. We just send usage signals to the scheduler so that is can adjust the fetch frequency. - Other? A first issue with the way we do that currently is that the general metrics recording should not be tied to a bridge instance. The metrics recording itself is done through plain functions. Event notifications themselves also doesn't need it as it is all related to the single content suggestions service that is accessible through the profile anyway. That would leave the SnippetsBridge with functions related to being a suggestions source and more 2 way communication between java and c++. The second issue is how some general suggestions metrics should not care about being in the NTP or not, so move out of NewTabPageUMA.
,
Apr 3 2017
I filed this thinking mostly about the java side of the recording, but if can improve also the c++ side it's even better!
,
May 22 2017
For java this was mostly fixed in issue 710268 and with https://codereview.chromium.org/2846233003 (issue 693593) Passing over to treib@ if there is something to do for C++/iOS
,
Jun 28 2017
,
Jun 28 2017
I think we're in a much better state now here, though two metrics (dismissals of suggestions and of categories) are still recorded from the bridge. I guess it should be easy to move those into the service - that should happen at least before we launch on iOS. All that said, I'm not very up to date with the latest on metrics. Over to Markus for that.
,
Sep 27 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by treib@chromium.org
, Apr 3 2017Labels: OS-iOS