Commit bb428089 authored by brettw@chromium.org's avatar brettw@chromium.org

Remove dev check for file chooser.

Disable the permissions check for dev interfaces in the PPAPI proxy. We don't
currently use this check for anything, and it's breaking integrated Flash. We
will need to work on getting the permissions correct.

BUG=138902

Review URL: https://chromiumcodereview.appspot.com/10855152

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151541 0039d316-1c4b-4281-b951-d872f2087c98
parent 23c5d295
......@@ -36,7 +36,11 @@ scoped_ptr<ResourceHost> ContentRendererPepperHostFactory::CreateResourceHost(
// Resources for dev interfaces.
// TODO(brettw) when we support any public or private interfaces, put them in
// a separate switch above.
if (GetPermissions().HasPermission(ppapi::PERMISSION_DEV)) {
// TODO(brettw) put back this dev check! This was removed to fix issue 138902
// where the permissions for bundled Flash (but not Flash that you specify
// on the command line, making it difficult to test) are incorrect.
/*if (GetPermissions().HasPermission(ppapi::PERMISSION_DEV))*/ {
switch (message.type()) {
case PpapiHostMsg_FileChooser_Create::ID:
return scoped_ptr<ResourceHost>(new PepperFileChooserHost(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment