Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(637)

Side by Side Diff: chrome/browser/installable/installable_metrics.h

Issue 2844383004: Split the NOT_STARTED and STOPPED_BEFORE_COMPLETION installability metrics. (Closed)
Patch Set: Fix comment Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « chrome/browser/installable/installable_manager_browsertest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_INSTALLABLE_INSTALLABLE_METRICS_H_ 5 #ifndef CHROME_BROWSER_INSTALLABLE_INSTALLABLE_METRICS_H_
6 #define CHROME_BROWSER_INSTALLABLE_INSTALLABLE_METRICS_H_ 6 #define CHROME_BROWSER_INSTALLABLE_INSTALLABLE_METRICS_H_
7 7
8 // This enum backs a UMA histogram and must be treated as append-only. 8 // This enum backs a UMA histogram and must be treated as append-only.
9 enum class InstallabilityCheckStatus { 9 enum class InstallabilityCheckStatus {
10 NOT_STARTED, 10 NOT_STARTED,
11 STOPPED_BEFORE_COMPLETION, 11 NOT_COMPLETED,
12 IN_PROGRESS_NON_PROGRESSIVE_WEB_APP, 12 IN_PROGRESS_NON_PROGRESSIVE_WEB_APP,
13 IN_PROGRESS_PROGRESSIVE_WEB_APP, 13 IN_PROGRESS_PROGRESSIVE_WEB_APP,
14 COMPLETE_NON_PROGRESSIVE_WEB_APP, 14 COMPLETE_NON_PROGRESSIVE_WEB_APP,
15 COMPLETE_PROGRESSIVE_WEB_APP, 15 COMPLETE_PROGRESSIVE_WEB_APP,
16 COUNT, 16 COUNT,
17 }; 17 };
18 18
19 class InstallableMetrics { 19 class InstallableMetrics {
20 public: 20 public:
21 static void RecordMenuOpenHistogram(InstallabilityCheckStatus status); 21 static void RecordMenuOpenHistogram(InstallabilityCheckStatus status);
22 static void RecordMenuItemAddToHomescreenHistogram( 22 static void RecordMenuItemAddToHomescreenHistogram(
23 InstallabilityCheckStatus status); 23 InstallabilityCheckStatus status);
24 }; 24 };
25 25
26 #endif // CHROME_BROWSER_INSTALLABLE_INSTALLABLE_METRICS_H_ 26 #endif // CHROME_BROWSER_INSTALLABLE_INSTALLABLE_METRICS_H_
OLDNEW
« no previous file with comments | « chrome/browser/installable/installable_manager_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698