Add GetURLForBrowserTab method to the external file system backend.

chrome/browser/chromeos/fileapi/file_system_backend.cc

 // Copyright 2013 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/fileapi/file_system_backend.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/chromeos/fileapi/file_access_permissions.h"
@@ skipping 369 matching lines @@
   return scoped_ptr<storage::FileStreamWriter>();
 }
 
 bool FileSystemBackend::GetVirtualPath(
     const base::FilePath& filesystem_path,
     base::FilePath* virtual_path) {
   return mount_points_->GetVirtualPath(filesystem_path, virtual_path) ||
          system_mount_points_->GetVirtualPath(filesystem_path, virtual_path);
 }
 
+void FileSystemBackend::GetURLForBrowserTab(
+    const storage::FileSystemURL& url,
+    const URLCallback& callback) {
+  switch (url.type()) {

[mtomasz, 2014/09/04 05:30:01]

I think we should add a DCHECK for url_is_valid(), and also check for
permissions by calling IsAccessAllowed(). Similarly to other methods.

[hirono, 2014/09/04 06:10:59]

Done.

+    case storage::kFileSystemTypeDrive:
+      drive_delegate_->GetURLForBrowserTab(url, callback);
+      return;
+    case storage::kFileSystemTypeProvided:
+      file_system_provider_delegate_->GetURLForBrowserTab(url, callback);
+      return;
+    case storage::kFileSystemTypeDeviceMediaAsFileStorage:
+      mtp_delegate_->GetURLForBrowserTab(url, callback);
+      return;
+    case storage::kFileSystemTypeNativeLocal:
+    case storage::kFileSystemTypeRestrictedNativeLocal:
+      callback.Run(GURL());
+      return;
+    default:
+      NOTREACHED();
+      callback.Run(GURL());
+      return;

[mtomasz, 2014/09/04 05:30:01]

In another switch-case statements in this file we call the default callback
outside the switch. Can we do that here too for consistency?

See: #316, #347, #380.

[hirono, 2014/09/04 06:10:59]

Done.

+  }
+}
+
 }  // namespace chromeos