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

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

Issue 371933002: Add UMA for the new generic permisison class (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rework the UMA based on mpearsons suggestion. Add extra UMA for ignored and dismissed requests Created 6 years, 5 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 | Annotate | Revision Log
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_CONTENT_SETTINGS_PERMISSION_CONTEXT_BASE_H_ 5 #ifndef CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_CONTEXT_BASE_H_
6 #define CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_CONTEXT_BASE_H_ 6 #define CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_CONTEXT_BASE_H_
7 7
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/containers/scoped_ptr_hash_map.h" 9 #include "base/containers/scoped_ptr_hash_map.h"
10 #include "base/memory/ref_counted.h" 10 #include "base/memory/ref_counted.h"
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 const GURL& requesting_origin, 72 const GURL& requesting_origin,
73 const GURL& embedder_origin, 73 const GURL& embedder_origin,
74 bool user_gesture, 74 bool user_gesture,
75 const BrowserPermissionCallback& callback); 75 const BrowserPermissionCallback& callback);
76 76
77 // Called when permission is granted without interactively asking the user. 77 // Called when permission is granted without interactively asking the user.
78 void PermissionDecided(const PermissionRequestID& id, 78 void PermissionDecided(const PermissionRequestID& id,
79 const GURL& requesting_origin, 79 const GURL& requesting_origin,
80 const GURL& embedder_origin, 80 const GURL& embedder_origin,
81 const BrowserPermissionCallback& callback, 81 const BrowserPermissionCallback& callback,
82 bool persist,
82 bool allowed); 83 bool allowed);
83 84
84 void NotifyPermissionSet(const PermissionRequestID& id, 85 void NotifyPermissionSet(const PermissionRequestID& id,
85 const GURL& requesting_origin, 86 const GURL& requesting_origin,
86 const GURL& embedder_origin, 87 const GURL& embedder_origin,
87 const BrowserPermissionCallback& callback, 88 const BrowserPermissionCallback& callback,
88 bool persist, 89 bool persist,
89 bool allowed); 90 bool allowed);
90 91
91 // Implementors can override this method to update the icons on the 92 // Implementors can override this method to update the icons on the
(...skipping 16 matching lines...) Expand all
108 109
109 Profile* profile_; 110 Profile* profile_;
110 const ContentSettingsType permission_type_; 111 const ContentSettingsType permission_type_;
111 base::WeakPtrFactory<PermissionContextBase> weak_factory_; 112 base::WeakPtrFactory<PermissionContextBase> weak_factory_;
112 scoped_ptr<PermissionQueueController> permission_queue_controller_; 113 scoped_ptr<PermissionQueueController> permission_queue_controller_;
113 base::ScopedPtrHashMap<std::string, PermissionBubbleRequest> 114 base::ScopedPtrHashMap<std::string, PermissionBubbleRequest>
114 pending_bubbles_; 115 pending_bubbles_;
115 }; 116 };
116 117
117 #endif // CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_CONTEXT_BASE_H_ 118 #endif // CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_CONTEXT_BASE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698