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

Side by Side Diff: chrome/browser/content_settings/chrome_content_settings_utils.h

Issue 2662163002: Add UMA histogram for popup actions on Android. (Closed)
Patch Set: Created 3 years, 10 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 (c) 2014 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2014 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_CONTENT_SETTINGS_CHROME_CONTENT_SETTINGS_UTILS_H_ 5 #ifndef CHROME_BROWSER_CONTENT_SETTINGS_CHROME_CONTENT_SETTINGS_UTILS_H_
6 #define CHROME_BROWSER_CONTENT_SETTINGS_CHROME_CONTENT_SETTINGS_UTILS_H_ 6 #define CHROME_BROWSER_CONTENT_SETTINGS_CHROME_CONTENT_SETTINGS_UTILS_H_
7 7
8 // Put utility functions only used by //chrome code here. If a function declared 8 // Put utility functions only used by //chrome code here. If a function declared
9 // here would be meaningfully shared with other platforms, consider moving it to 9 // here would be meaningfully shared with other platforms, consider moving it to
10 // components/content_settings/core/browser/content_settings_utils.h. 10 // components/content_settings/core/browser/content_settings_utils.h.
(...skipping 29 matching lines...) Expand all
40 40
41 // UMA histogram for actions that a user can perform on the pop-up blocked page 41 // UMA histogram for actions that a user can perform on the pop-up blocked page
42 // action in the omnibox. The enum values correspond to histogram entries, so do 42 // action in the omnibox. The enum values correspond to histogram entries, so do
43 // not remove any existing values. 43 // not remove any existing values.
44 enum PopupsAction { 44 enum PopupsAction {
45 POPUPS_ACTION_DISPLAYED_BLOCKED_ICON_IN_OMNIBOX = 0, 45 POPUPS_ACTION_DISPLAYED_BLOCKED_ICON_IN_OMNIBOX = 0,
46 POPUPS_ACTION_DISPLAYED_BUBBLE, 46 POPUPS_ACTION_DISPLAYED_BUBBLE,
47 POPUPS_ACTION_SELECTED_ALWAYS_ALLOW_POPUPS_FROM, 47 POPUPS_ACTION_SELECTED_ALWAYS_ALLOW_POPUPS_FROM,
48 POPUPS_ACTION_CLICKED_LIST_ITEM_CLICKED, 48 POPUPS_ACTION_CLICKED_LIST_ITEM_CLICKED,
49 POPUPS_ACTION_CLICKED_MANAGE_POPUPS_BLOCKING, 49 POPUPS_ACTION_CLICKED_MANAGE_POPUPS_BLOCKING,
50 POPUPS_ACTION_DISPLAYED_INFOBAR_ON_MOBILE,
51 POPUPS_ACTION_CLICKED_ALWAYS_SHOW_ON_MOBILE,
50 POPUPS_ACTION_COUNT 52 POPUPS_ACTION_COUNT
51 }; 53 };
52 54
53 void RecordPopupsAction(PopupsAction action); 55 void RecordPopupsAction(PopupsAction action);
54 56
55 } // namespace content_settings 57 } // namespace content_settings
56 58
57 #endif // CHROME_BROWSER_CONTENT_SETTINGS_CHROME_CONTENT_SETTINGS_UTILS_H_ 59 #endif // CHROME_BROWSER_CONTENT_SETTINGS_CHROME_CONTENT_SETTINGS_UTILS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698