New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 742264 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Not on Chrome anymore
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocking:
issue 746274



Sign in to add a comment

Report Tile impressions when they are shown, not when they are fetched

Project Member Reported by dgn@chromium.org, Jul 13 2017

Issue description

We currently record tile impressions when we finish fetching the tiles and their icons, but this is independent from when we show them. On small screens, we fetch more tiles than we can show, and we just clip the list. 

As a result, we would report showing 8 tiles to UMA even though the layout has space for only 6 and never shows the last 2.

We could let tiles report when they are shown after they are attached to the view for the first time as a fix.

Targeting M-62 as this would be likely worked on as part of the zine-explore refactoring
 

Comment 1 by dgn@chromium.org, Jul 13 2017

Description: Show this description

Comment 2 by dgn@chromium.org, Jul 19 2017

Blocking: 746274

Comment 3 by dgn@chromium.org, Jul 19 2017

Owner: galinap@google.com
galinap@ could you please have a look at this?

Comment 4 by fi...@chromium.org, Jul 20 2017

Labels: zine-triaged

Comment 5 by galinap@google.com, Aug 30 2017

Labels: -Pri-2 Pri-3

Comment 6 by galinap@google.com, Sep 15 2017

Owner: dgn@chromium.org
Labels: -M-62 M-63
Why is this P3? Sounds fairly important. I guess this isn't targeted for M-62 anymore?

Comment 8 by dgn@chromium.org, Sep 28 2017

Labels: -M-63 M-64
Not 63 either. I would like to tackle it as part of cleaning up the code, but it's not a request from owners of the metrics, hence the P3. (good enough, yadda, yadda)

Comment 9 by dgn@chromium.org, Dec 5 2017

Status: WontFix (was: Assigned)
Maybe to be revisited post jardin, if there is interest

Sign in to add a comment