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

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

Issue 2178833002: Add new app banner metrics using InstallableStatusCode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@banner-integrate-checker-no-refptr
Patch Set: Fix histogram name Created 4 years, 4 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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_LOGGING_H_ 5 #ifndef CHROME_BROWSER_INSTALLABLE_INSTALLABLE_LOGGING_H_
6 #define CHROME_BROWSER_INSTALLABLE_INSTALLABLE_LOGGING_H_ 6 #define CHROME_BROWSER_INSTALLABLE_INSTALLABLE_LOGGING_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 namespace content { 10 namespace content {
11 class WebContents; 11 class WebContents;
12 } 12 }
13 13
14 // These values are a central reference for installability errors. The 14 // These values are a central reference for installability errors. The
15 // InstallableManager will specify an InstallableErrorCode (or 15 // InstallableManager will specify an InstallableStatusCode (or
16 // NO_ERROR_DETECTED) in its result. Clients may also add their own error codes 16 // NO_ERROR_DETECTED) in its result. Clients may also add their own error codes,
17 // to this list, and utilise LogErrorToConsole to write a message to the 17 // and utilise LogErrorToConsole to write a message to the devtools console.
18 // devtools console. 18 // This enum backs an UMA histogram, so it must be treated as append-only.
19 // TODO(dominickn) add an UMA metric for this enum. 19 enum InstallableStatusCode {
20 enum InstallableErrorCode {
21 NO_ERROR_DETECTED, 20 NO_ERROR_DETECTED,
22 RENDERER_EXITING, 21 RENDERER_EXITING,
23 RENDERER_CANCELLED, 22 RENDERER_CANCELLED,
24 USER_NAVIGATED, 23 USER_NAVIGATED,
25 NOT_IN_MAIN_FRAME, 24 NOT_IN_MAIN_FRAME,
26 NOT_FROM_SECURE_ORIGIN, 25 NOT_FROM_SECURE_ORIGIN,
27 NO_MANIFEST, 26 NO_MANIFEST,
28 MANIFEST_EMPTY, 27 MANIFEST_EMPTY,
29 START_URL_NOT_VALID, 28 START_URL_NOT_VALID,
30 MANIFEST_MISSING_NAME_OR_SHORT_NAME, 29 MANIFEST_MISSING_NAME_OR_SHORT_NAME,
31 MANIFEST_DISPLAY_NOT_SUPPORTED, 30 MANIFEST_DISPLAY_NOT_SUPPORTED,
32 MANIFEST_MISSING_SUITABLE_ICON, 31 MANIFEST_MISSING_SUITABLE_ICON,
33 NO_MATCHING_SERVICE_WORKER, 32 NO_MATCHING_SERVICE_WORKER,
34 NO_ACCEPTABLE_ICON, 33 NO_ACCEPTABLE_ICON,
35 CANNOT_DOWNLOAD_ICON, 34 CANNOT_DOWNLOAD_ICON,
36 NO_ICON_AVAILABLE, 35 NO_ICON_AVAILABLE,
37 PLATFORM_NOT_SUPPORTED_ON_ANDROID, 36 PLATFORM_NOT_SUPPORTED_ON_ANDROID,
38 NO_ID_SPECIFIED, 37 NO_ID_SPECIFIED,
39 IDS_DO_NOT_MATCH, 38 IDS_DO_NOT_MATCH,
39 ALREADY_INSTALLED,
40 INSUFFICIENT_ENGAGEMENT,
41 PACKAGE_NAME_OR_START_URL_EMPTY,
42 PREVIOUSLY_BLOCKED,
43 PREVIOUSLY_IGNORED,
44 SHOWING_NATIVE_APP_BANNER,
45 SHOWING_WEB_APP_BANNER,
46 FAILED_TO_CREATE_BANNER,
40 MAX_ERROR_CODE, 47 MAX_ERROR_CODE,
41 }; 48 };
42 49
43 // Logs a message associated with |code| with a parameter |param| to the 50 // Logs a message associated with |code| with a parameter |param| to the
44 // devtools console attached to |web_contents|. Does nothing if |web_contents| 51 // devtools console attached to |web_contents|. Does nothing if |web_contents|
45 // is nullptr. 52 // is nullptr.
46 void LogErrorToConsole(content::WebContents* web_contents, 53 void LogErrorToConsole(content::WebContents* web_contents,
47 InstallableErrorCode code, 54 InstallableStatusCode code,
48 const std::string& param); 55 const std::string& param);
49 56
50 #endif // CHROME_BROWSER_INSTALLABLE_INSTALLABLE_LOGGING_H_ 57 #endif // CHROME_BROWSER_INSTALLABLE_INSTALLABLE_LOGGING_H_
OLDNEW
« no previous file with comments | « chrome/browser/banners/app_banner_settings_helper_unittest.cc ('k') | chrome/browser/installable/installable_logging.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698