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

Side by Side Diff: content/public/common/browser_plugin_message_enums.h

Issue 21297005: <webview>: Refactor Permission API to chrome (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@cleanup_permissions
Patch Set: Addressed jam@'s comments Created 7 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
jam 2013/08/09 00:07:27 nit: rename this file to browser_plugin_permission
Fady Samuel 2013/08/09 15:52:58 Done.
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 CONTENT_COMMON_BROWSER_PLUGIN_BROWSER_PLUGIN_MESSAGE_ENUMS_H_ 5 #ifndef CONTENT_COMMON_BROWSER_PLUGIN_MESSAGE_ENUMS_H_
6 #define CONTENT_COMMON_BROWSER_PLUGIN_BROWSER_PLUGIN_MESSAGE_ENUMS_H_ 6 #define CONTENT_COMMON_BROWSER_PLUGIN_MESSAGE_ENUMS_H_
7 7
8 enum BrowserPluginPermissionType { 8 enum BrowserPluginPermissionType {
9 // Unknown type of permission request. 9 // Unknown type of permission request.
10 BrowserPluginPermissionTypeUnknown, 10 BROWSER_PLUGIN_PERMISSION_TYPE_UNKNOWN,
11 11
12 // Download. 12 // Download.
jam 2013/08/09 00:07:27 nit: this, line 15, and line 21 should be taken ou
Fady Samuel 2013/08/09 15:52:58 Done.
13 BrowserPluginPermissionTypeDownload, 13 BROWSER_PLUGIN_PERMISSION_TYPE_DOWNLOAD,
14 14
15 // Geolocation. 15 // Geolocation.
16 BrowserPluginPermissionTypeGeolocation, 16 BROWSER_PLUGIN_PERMISSION_TYPE_GEOLOCATION,
17 17
18 // Media access (audio/video) permission request type. 18 // Media access (audio/video) permission request type.
19 BrowserPluginPermissionTypeMedia, 19 BROWSER_PLUGIN_PERMISSION_TYPE_MEDIA,
20 20
21 // PointerLock 21 // PointerLock
22 BrowserPluginPermissionTypePointerLock, 22 BROWSER_PLUGIN_PERMISSION_TYPE_POINTER_LOCK,
23 23
24 24
jam 2013/08/09 00:07:27 nit: extra line
Fady Samuel 2013/08/09 15:52:58 Done.
25 // New window requests. 25 // New window requests.
26 // Note: Even though new windows don't use the permission API, the new window 26 // Note: Even though new windows don't use the permission API, the new window
27 // API is sufficiently similar that it's convenient to consider it a 27 // API is sufficiently similar that it's convenient to consider it a
28 // permission type for code reuse. 28 // permission type for code reuse.
29 BrowserPluginPermissionTypeNewWindow, 29 BROWSER_PLUGIN_PERMISSION_TYPE_NEW_WINDOW,
30 30
31 // JavaScript Dialogs: prompt, alert, confirm 31 // JavaScript Dialogs: prompt, alert, confirm
32 // Note: Even through dialogs do not use the permission API, the dialog API 32 // Note: Even through dialogs do not use the permission API, the dialog API
33 // is sufficiently similiar that it's convenient to consider it a permission 33 // is sufficiently similiar that it's convenient to consider it a permission
34 // type for code reuse. 34 // type for code reuse.
35 BrowserPluginPermissionTypeJavaScriptDialog 35 BROWSER_PLUGIN_PERMISSION_TYPE_JAVASCRIPT_DIALOG
36 }; 36 };
37 37
38 #endif // CONTENT_COMMON_BROWSER_PLUGIN_BROWSER_PLUGIN_MESSAGE_ENUMS_H_ 38 #endif // CONTENT_COMMON_BROWSER_PLUGIN_MESSAGE_ENUMS_H_
OLDNEW
« no previous file with comments | « content/public/browser/browser_plugin_guest_delegate.cc ('k') | content/renderer/browser_plugin/browser_plugin.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698