| commit | 98844a9fa47bea1932f2b38083648d0279c0fe0e | [log] [tgz] |
|---|---|---|
| author | Alan Cutter <[email protected]> | Wed Jun 19 21:11:40 2019 |
| committer | Commit Bot <[email protected]> | Wed Jun 19 21:11:40 2019 |
| tree | a2989a32bbd8213058064dc248880d80b21b6598 | |
| parent | a8504d98e9187dc8df59f04d8292a76f812a4475 [diff] [blame] |
Measure user actions "Menu > Install PWA/Create shortcut" separately Before this CL we were measuring "Menu > Install PWA..." and "Menu > More tools > Create shortcut..." under the same user action "CreateHostedApp". This CL splits them into "MenuInstallWebApp" and "CreateShortcut". Bug: 910016 Change-Id: I25c2bd801a7a429499d0693c6dfc0937b707dab7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1666322 Auto-Submit: Alan Cutter <[email protected]> Commit-Queue: Brian White <[email protected]> Reviewed-by: Peter Kasting <[email protected]> Reviewed-by: Brian White <[email protected]> Reviewed-by: Raymes Khoury <[email protected]> Cr-Commit-Position: refs/heads/master@{#670644}
diff --git a/chrome/browser/ui/browser_command_controller.cc b/chrome/browser/ui/browser_command_controller.cc index e4773b83..784bf153 100644 --- a/chrome/browser/ui/browser_command_controller.cc +++ b/chrome/browser/ui/browser_command_controller.cc
@@ -612,10 +612,12 @@ browser_->OpenFile(); break; case IDC_CREATE_SHORTCUT: + base::RecordAction(base::UserMetricsAction("CreateShortcut")); CreateBookmarkAppFromCurrentWebContents(browser_, true /* force_shortcut_app */); break; case IDC_INSTALL_PWA: + base::RecordAction(base::UserMetricsAction("InstallWebAppFromMenu")); CreateBookmarkAppFromCurrentWebContents(browser_, false /* force_shortcut_app */); break;