Add oem mount point to cros_mount_provider.

chrome/browser/chromeos/extensions/file_handler_util.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/extensions/file_handler_util.h"
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/i18n/case_conversion.h"
 #include "base/json/json_writer.h"
@@ skipping 39 matching lines @@
 using extensions::Extension;
 
 namespace file_handler_util {
 
 const char kTaskFile[] = "file";
 const char kTaskDrive[] = "drive";
 const char kTaskWebIntent[] = "web-intent";
 
 namespace {
 
+typedef std::set<const FileBrowserHandler*> FileBrowserHandlerSet;
+
 // Legacy Drive task extension prefix, used by CrackTaskID.
 const char kDriveTaskExtensionPrefix[] = "drive-app:";
 const size_t kDriveTaskExtensionPrefixLength =
     arraysize(kDriveTaskExtensionPrefix) - 1;
 
-typedef std::set<const FileBrowserHandler*> FileBrowserHandlerSet;
+const char kOemMountPointID[] = "oem";
 
 const int kReadWriteFilePermissions = base::PLATFORM_FILE_OPEN |
                                       base::PLATFORM_FILE_CREATE |
                                       base::PLATFORM_FILE_OPEN_ALWAYS |
                                       base::PLATFORM_FILE_CREATE_ALWAYS |
                                       base::PLATFORM_FILE_OPEN_TRUNCATED |
                                       base::PLATFORM_FILE_READ |
                                       base::PLATFORM_FILE_WRITE |
                                       base::PLATFORM_FILE_EXCLUSIVE_READ |
                                       base::PLATFORM_FILE_EXCLUSIVE_WRITE |
@@ skipping 636 matching lines @@
     if (!handler_extension_.get())
       return false;
 
     if (handler_pid_ == 0)
       return false;
 
     fileapi::FileSystemURL url(origin_file_url);
     if (!chromeos::CrosMountPointProvider::CanHandleURL(url))
       return false;
 
+    if (url.filesystem_id() == kOemMountPointID) {

[zel, 2012/09/27 20:11:10]

let's create a more general concept of read-only mount points than having
one-off checks like this. ideally this info should reside with the mount point
provider

+      LOG(ERROR) << "Can't grant permissions for oem mount point.";
+      return false;
+    }
+
     fileapi::ExternalFileSystemMountPointProvider* external_provider =
         file_system_context_->external_provider();
     if (!external_provider || !external_provider->IsAccessAllowed(url))
       return false;
 
     // Make sure this url really being used by the right caller extension.
     if (source_url_.GetOrigin() != url.origin()) {
       DidFail(base::PLATFORM_FILE_ERROR_SECURITY);
       return false;
     }
@@ skipping 306 matching lines @@
     done.Run(success);
 
   return true;
 }
 
 bool WebIntentTaskExecutor::ExecuteForURL(const GURL& file_url) {
   fileapi::FileSystemURL url(file_url);
   if (!chromeos::CrosMountPointProvider::CanHandleURL(url))
     return false;
 
+  if (url.filesystem_id() == kOemMountPointID) {
+    LOG(ERROR) << "Can't grant permissions for oem mount point.";
+    return false;
+  }
+
   scoped_refptr<fileapi::FileSystemContext> file_system_context =
       BrowserContext::GetDefaultStoragePartition(profile())->
       GetFileSystemContext();
   fileapi::ExternalFileSystemMountPointProvider* external_provider =
       file_system_context->external_provider();
   if (!external_provider || !external_provider->IsAccessAllowed(url))
     return false;
 
   // Make sure this url really being used by the right caller extension.
   if (source_url_.GetOrigin() != url.origin())
     return false;
 
   FilePath local_path = url.path();
   extensions::LaunchPlatformAppWithPath(profile(), GetExtension(), local_path);
   return true;
 }
 
 } // namespace file_handler_util