Changed FileSystemPointProvider::IsAcccessAllowed() to take a single FileSystemURL instead of a tri…

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 558 matching lines @@
 
     fileapi::FileSystemURL url(origin_file_url);
     if (!chromeos::CrosMountPointProvider::CanHandleURL(url))
       return false;
 
     fileapi::ExternalFileSystemMountPointProvider* external_provider =
         file_system_context_->external_provider();
     if (!external_provider)
       return false;
 
-    if (!external_provider->IsAccessAllowed(url.origin(),
-                                            url.type(),
-                                            url.virtual_path())) {
+    if (!external_provider->IsAccessAllowed(url))

[hashimoto, 2012/08/27 06:43:22]

nit: Combine this 'if' with the above one using '||'.

       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;
     }
 
     // Check if this file system entry exists first.
     base::PlatformFileInfo file_info;
 
@@ skipping 284 matching lines @@
         handler_pid,
         handler_host_permissions_[i].first,
         handler_host_permissions_[i].second);
   }
 
   // We don't need this anymore.
   handler_host_permissions_.clear();
 }
 
 } // namespace file_handler_util