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

Unified Diff: content/public/browser/render_view_host.h

Issue 18129002: Update the child process security policy to use explicit permission grants. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: content/public/browser/render_view_host.h
diff --git a/content/public/browser/render_view_host.h b/content/public/browser/render_view_host.h
index ecb2fb712d7f3cc4b75274aece56780bf2dbde40..12ced4c2758c903d311b12a5c995fb3bf50f6bf3 100644
--- a/content/public/browser/render_view_host.h
+++ b/content/public/browser/render_view_host.h
@@ -209,13 +209,17 @@ class CONTENT_EXPORT RenderViewHost : virtual public RenderWidgetHost {
// closing the entire tab.
virtual void FirePageBeforeUnload(bool for_cross_site_transition) = 0;
+ enum FileSelectionPermissions {
vandebo (ex-Chrome) 2013/06/27 22:35:17 style guide says this should go above any methods.
Greg Billock 2013/06/28 18:40:33 Yeah, that was what I was thinking. I can move it
+ FILE_PERMISSION_READ_ONLY,
+ FILE_PERMISSION_WRITE
+ };
+
// Notifies the Listener that one or more files have been chosen by the user
- // from a file chooser dialog for the form. |permissions| are flags from the
- // base::PlatformFileFlags enum which specify which file permissions should
- // be granted to the renderer.
+ // from a file chooser dialog for the form. |permissions| is the permission
+ // security policy for the selected file.
virtual void FilesSelectedInChooser(
const std::vector<ui::SelectedFileInfo>& files,
- int permissions) = 0;
+ FileSelectionPermissions permissions) = 0;
virtual RenderViewHostDelegate* GetDelegate() const = 0;

Powered by Google App Engine
This is Rietveld 408576698