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

Side by Side Diff: chrome/browser/permissions/permission_context_uma_util.h

Issue 1388343003: Adds a field trial to control Web API kill switches. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review Comments Created 5 years, 2 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 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_PERMISSIONS_PERMISSION_CONTEXT_UMA_UTIL_H_ 5 #ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_UMA_UTIL_H_
6 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_UMA_UTIL_H_ 6 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_UMA_UTIL_H_
7 7
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "components/content_settings/core/common/content_settings_types.h" 9 #include "components/content_settings/core/common/content_settings_types.h"
10 10
11 class GURL; 11 class GURL;
12 class Profile; 12 class Profile;
13 13
14 // Provides a convenient way of logging UMA for permission related 14 // Provides a convenient way of logging UMA for permission related
15 // operations. 15 // operations.
16 class PermissionContextUmaUtil { 16 class PermissionContextUmaUtil {
17 public: 17 public:
18 static void PermissionRequested(ContentSettingsType permission, 18 static void PermissionRequested(ContentSettingsType permission,
19 const GURL& requesting_origin, 19 const GURL& requesting_origin,
20 const GURL& embedding_origin, 20 const GURL& embedding_origin,
21 Profile* profile); 21 Profile* profile);
22 static void PermissionGranted(ContentSettingsType permission, 22 static void PermissionGranted(ContentSettingsType permission,
23 const GURL& requesting_origin); 23 const GURL& requesting_origin);
24 static void PermissionDenied(ContentSettingsType permission, 24 static void PermissionDenied(ContentSettingsType permission,
25 const GURL& requesting_origin); 25 const GURL& requesting_origin);
26 static void PermissionDismissed(ContentSettingsType permission, 26 static void PermissionDismissed(ContentSettingsType permission,
27 const GURL& requesting_origin); 27 const GURL& requesting_origin);
28 static void PermissionIgnored(ContentSettingsType permission, 28 static void PermissionIgnored(ContentSettingsType permission,
29 const GURL& requesting_origin); 29 const GURL& requesting_origin);
30 static const std::string GetPermissionString(ContentSettingsType permission);
mlamouri (slow - plz ping) 2015/10/09 12:07:54 I don't think this needs to be part of the class.
kcarattini 2015/10/12 03:58:06 No, that's why it is static. None of the other met
30 31
31 private: 32 private:
32 DISALLOW_IMPLICIT_CONSTRUCTORS(PermissionContextUmaUtil); 33 DISALLOW_IMPLICIT_CONSTRUCTORS(PermissionContextUmaUtil);
33 }; 34 };
34 35
35 #endif // CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_UMA_UTIL_H_ 36 #endif // CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_UMA_UTIL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698