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

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

Issue 1164073005: Allow script to request durable storage permission (chrome side) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix conflicts after midi constant permission landed Created 5 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 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 #include "chrome/browser/permissions/permission_context_uma_util.h" 5 #include "chrome/browser/permissions/permission_context_uma_util.h"
6 6
7 #include "base/metrics/histogram_macros.h" 7 #include "base/metrics/histogram_macros.h"
8 #include "base/strings/stringprintf.h" 8 #include "base/strings/stringprintf.h"
9 #include "chrome/browser/browser_process.h" 9 #include "chrome/browser/browser_process.h"
10 #include "chrome/browser/permissions/permission_manager.h" 10 #include "chrome/browser/permissions/permission_manager.h"
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 #if defined(OS_ANDROID) || defined(OS_CHROMEOS) 124 #if defined(OS_ANDROID) || defined(OS_CHROMEOS)
125 case CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER: 125 case CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER:
126 PERMISSION_ACTION_UMA( 126 PERMISSION_ACTION_UMA(
127 secure_origin, 127 secure_origin,
128 "ContentSettings.PermissionActions_ProtectedMedia", 128 "ContentSettings.PermissionActions_ProtectedMedia",
129 "ContentSettings.PermissionActionsSecureOrigin_ProtectedMedia", 129 "ContentSettings.PermissionActionsSecureOrigin_ProtectedMedia",
130 "ContentSettings.PermissionActionsInsecureOrigin_ProtectedMedia", 130 "ContentSettings.PermissionActionsInsecureOrigin_ProtectedMedia",
131 action); 131 action);
132 break; 132 break;
133 #endif 133 #endif
134 case CONTENT_SETTINGS_TYPE_DURABLE_STORAGE:
135 PERMISSION_ACTION_UMA(
136 secure_origin, "ContentSettings.PermissionActions_DurableStorage",
137 "ContentSettings.PermissionActionsSecureOrigin_DurableStorage",
138 "ContentSettings.PermissionActionsInsecureOrigin_DurableStorage",
139 action);
140 break;
134 default: 141 default:
135 NOTREACHED() << "PERMISSION " << permission << " not accounted for"; 142 NOTREACHED() << "PERMISSION " << permission << " not accounted for";
136 } 143 }
137 144
138 const std::string& rappor_metric = GetRapporMetric(permission, action); 145 const std::string& rappor_metric = GetRapporMetric(permission, action);
139 if (!rappor_metric.empty()) 146 if (!rappor_metric.empty())
140 rappor::SampleDomainAndRegistryFromGURL( 147 rappor::SampleDomainAndRegistryFromGURL(
141 g_browser_process->rappor_service(), rappor_metric, 148 g_browser_process->rappor_service(), rappor_metric,
142 requesting_origin); 149 requesting_origin);
143 } 150 }
144 151
145 std::string PermissionTypeToString(PermissionType permission_type) { 152 std::string PermissionTypeToString(PermissionType permission_type) {
146 switch (permission_type) { 153 switch (permission_type) {
147 case PermissionType::MIDI_SYSEX: 154 case PermissionType::MIDI_SYSEX:
148 return "MidiSysex"; 155 return "MidiSysex";
149 case PermissionType::PUSH_MESSAGING: 156 case PermissionType::PUSH_MESSAGING:
150 return "PushMessaging"; 157 return "PushMessaging";
151 case PermissionType::NOTIFICATIONS: 158 case PermissionType::NOTIFICATIONS:
152 return "Notifications"; 159 return "Notifications";
153 case PermissionType::GEOLOCATION: 160 case PermissionType::GEOLOCATION:
154 return "Geolocation"; 161 return "Geolocation";
155 case PermissionType::PROTECTED_MEDIA_IDENTIFIER: 162 case PermissionType::PROTECTED_MEDIA_IDENTIFIER:
156 return "ProtectedMediaIdentifier"; 163 return "ProtectedMediaIdentifier";
164 case PermissionType::DURABLE_STORAGE:
165 return "DurableStorage";
157 case PermissionType::MIDI: 166 case PermissionType::MIDI:
158 return "Midi"; 167 return "Midi";
159 case PermissionType::NUM: 168 case PermissionType::NUM:
160 break; 169 break;
161 } 170 }
162 NOTREACHED(); 171 NOTREACHED();
163 return std::string(); 172 return std::string();
164 } 173 }
165 174
166 void RecordPermissionRequest(ContentSettingsType permission, 175 void RecordPermissionRequest(ContentSettingsType permission,
(...skipping 21 matching lines...) Expand all
188 type = PermissionType::MIDI_SYSEX; 197 type = PermissionType::MIDI_SYSEX;
189 break; 198 break;
190 case CONTENT_SETTINGS_TYPE_PUSH_MESSAGING: 199 case CONTENT_SETTINGS_TYPE_PUSH_MESSAGING:
191 type = PermissionType::PUSH_MESSAGING; 200 type = PermissionType::PUSH_MESSAGING;
192 break; 201 break;
193 #if defined(OS_ANDROID) || defined(OS_CHROMEOS) 202 #if defined(OS_ANDROID) || defined(OS_CHROMEOS)
194 case CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER: 203 case CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER:
195 type = PermissionType::PROTECTED_MEDIA_IDENTIFIER; 204 type = PermissionType::PROTECTED_MEDIA_IDENTIFIER;
196 break; 205 break;
197 #endif 206 #endif
207 case CONTENT_SETTINGS_TYPE_DURABLE_STORAGE:
208 type = content::PermissionType::DURABLE_STORAGE;
209 break;
198 default: 210 default:
199 NOTREACHED() << "PERMISSION " << permission << " not accounted for"; 211 NOTREACHED() << "PERMISSION " << permission << " not accounted for";
200 return; 212 return;
201 } 213 }
202 UMA_HISTOGRAM_ENUMERATION( 214 UMA_HISTOGRAM_ENUMERATION(
203 "ContentSettings.PermissionRequested", 215 "ContentSettings.PermissionRequested",
204 static_cast<base::HistogramBase::Sample>(type), 216 static_cast<base::HistogramBase::Sample>(type),
205 static_cast<base::HistogramBase::Sample>(PermissionType::NUM)); 217 static_cast<base::HistogramBase::Sample>(PermissionType::NUM));
206 if (secure_origin) { 218 if (secure_origin) {
207 UMA_HISTOGRAM_ENUMERATION( 219 UMA_HISTOGRAM_ENUMERATION(
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
265 277
266 void PermissionContextUmaUtil::PermissionDismissed( 278 void PermissionContextUmaUtil::PermissionDismissed(
267 ContentSettingsType permission, const GURL& requesting_origin) { 279 ContentSettingsType permission, const GURL& requesting_origin) {
268 RecordPermissionAction(permission, DISMISSED, requesting_origin); 280 RecordPermissionAction(permission, DISMISSED, requesting_origin);
269 } 281 }
270 282
271 void PermissionContextUmaUtil::PermissionIgnored( 283 void PermissionContextUmaUtil::PermissionIgnored(
272 ContentSettingsType permission, const GURL& requesting_origin) { 284 ContentSettingsType permission, const GURL& requesting_origin) {
273 RecordPermissionAction(permission, IGNORED, requesting_origin); 285 RecordPermissionAction(permission, IGNORED, requesting_origin);
274 } 286 }
OLDNEW
« no previous file with comments | « chrome/browser/permissions/permission_context.cc ('k') | chrome/browser/permissions/permission_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698