New issue
Advanced search Search tips

Issue 773160 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Task



Sign in to add a comment

TabUsageRecorder::RecordPageLoadStart is not called if a page is loaded via "Open In New Tab" context menu

Project Member Reported by eugene...@chromium.org, Oct 10 2017

Issue description

Steps to reproduce: 
1. Load a wikipedia page
2. Long press on any link
3. Tap Open In New Tab

Observed behavior: 
RecordPageLoadStart is not called for a child page

Expected behavior: 
RecordPageLoadStart should be called for a child page

 
Cc: gch...@chromium.org
gchatz@, do you know what is the implication of undercounting page loads?

Comment 2 by gch...@chromium.org, Oct 10 2017

1. Makes our crash rate higher than it really is.
2. If someone were to use page counts as a metric for user engagement, it would be undercounted.

Comment 3 by sczs@chromium.org, Oct 11 2017

Components: Internals>Metrics
Owner: olivierrobin@chromium.org
Status: Assigned (was: Untriaged)
Assigning to olivierrobin@ since he seems to be the owner, though I'm not sure if gchatz@ should be the owner since he's been working on metrics.
Cc: olivierrobin@chromium.org
Labels: -Type-Bug -Pri-2 Pri-3 Type-Task
Owner: ----
Status: Available (was: Assigned)
Per comment #2 the bug has pretty low implications, but it will be hard to fix the issue, because page loads are expected to be comparable across different versions of chrome. Will mark the bug Available.
Components: -Internals>Metrics
Labels: Hotlist-Metrics
FYI, we're switching over to Hotlist-Metrics for tracking specific metrics, as opposed to core metrics team work.  Thanks!
Components: Internals
Project Member

Comment 7 by sheriffbot@chromium.org, Oct 12

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: -gch...@chromium.org
Status: Available (was: Untriaged)
Still a problem. We have a code, which intentionally skips RecordPageLoadStart for child windows, but it's unclear why we have that code in ShouldRecordPageLoadStartForNavigation.

Sign in to add a comment