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

Issue 682216 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Bug



Sign in to add a comment

Add user action when opening the reading list

Project Member Reported by gambard@chromium.org, Jan 18 2017

Issue description

When the user opens the reading list, a bunch of histograms are logged but no user action.
I am asking this question again because almost all toolbar menu actions are logging a UserAction. It would be consistent to add one.
mardini, as the main user of the metrics, do you think we should add one?
 
Yes. Thank you, Gauthier. Good suggestion. 
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/03d984c79b0943e27f6b1239689a15b4830f4fd3

commit 03d984c79b0943e27f6b1239689a15b4830f4fd3
Author: gambard <gambard@chromium.org>
Date: Thu Jan 19 08:30:11 2017

Add UserAction when the user opens ReadingList

This CL adds the MobileMenuReadingList User Actions, logged when the user
presses the reading list button in the toolbar menu.

BUG= 682216 

Review-Url: https://codereview.chromium.org/2642763002
Cr-Commit-Position: refs/heads/master@{#444682}

[modify] https://crrev.com/03d984c79b0943e27f6b1239689a15b4830f4fd3/ios/chrome/browser/ui/tools_menu/tools_popup_controller.mm
[modify] https://crrev.com/03d984c79b0943e27f6b1239689a15b4830f4fd3/tools/metrics/actions/actions.xml

Status: Fixed (was: Assigned)

Sign in to add a comment