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

Issue 674405 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Task

Blocked on:
issue 624410



Sign in to add a comment

Request: Logging for Open Bookmark when on New Tab Page

Project Member Reported by mpear...@chromium.org, Dec 15 2016

Issue description


I'm thinking about getting a better view on how people use the NTP on desktop platforms.

We have the user action ClickedBookmarkBarURLButton.  And we have the histogram Bookmarks.LaunchLocation.  But neither of these helps me answer the question I have which is how many times do users click on bookmarks while on the new tab page.

 

Comment 1 by sky@chromium.org, Dec 15 2016

Indeed. Detached is only applicable to the ntp, but some percentage of attached is on the ntp.
Components: UI>Browser>NewTabPage
NTP folks, is there a way to answer this question nowadays?

Comment 3 by treib@chromium.org, May 9 2017

Cc: treib@chromium.org
I'm not aware of any existing metrics that would capture this. Shouldn't be too hard to add though.
Labels: -Type-Bug Type-Task
Status: Available (was: Untriaged)
NextAction: 2018-03-03
Who owns the (new) NTP efforts these days?
Owner: twelling...@chromium.org
Note that this request is about desktop.

Comment 8 by zea@chromium.org, Feb 1 2018

Owner: ramyan@chromium.org
+ramyan is ramping up on Desktop NTP ownership
NextAction: ----
Status: Started (was: Available)
It looks like I'll be able to do this by checking IsInstantNTP(WebContents). 
Ah, one more thing I thought of: IsInstantNTP will not cover the Incognito and Guest Mode NTPs. I don't know if that's good or bad in this case :)
For my original use case, I would prefer if the NTP covered all possible NTPs, instant or not, incognito or not, Google default search engine or not, guest mode, etc.

If it's hard to get them all (per bug 624410), then I can be satisfied with IsInstantNTP.

Can we at least do
IsInstantNTP() || url.spec() == chrome::kChromeUINewTabURL
?
(the latter will help capture things like extension-replaced NTPs)

Also, will that second term end up including incognito?
IsInstantNTP already covers third-party NTPs. Adding the comparison to kChromeUINewTabURL will also capture Incognito and Guest. I don't know about extensions.
Owner: ----
Status: Available (was: Started)
Making this available in case someone else has cycles to work on it in the near term. WIP crrev.com/r/908049 may have some useful comments to refer to when starting on this.
Blockedon: 624410

Sign in to add a comment