Add GetURLForBrowserTab method to the external file system backend.

chrome/browser/chromeos/drive/fileapi/file_system_backend_delegate.cc

 // Copyright 2014 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/drive/fileapi/file_system_backend_delegate.h"
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/memory/scoped_ptr.h"
+#include "chrome/browser/chromeos/drive/file_system_interface.h"
 #include "chrome/browser/chromeos/drive/file_system_util.h"
 #include "chrome/browser/chromeos/drive/fileapi/async_file_util.h"
 #include "chrome/browser/chromeos/drive/fileapi/fileapi_worker.h"
 #include "chrome/browser/chromeos/drive/fileapi/webkit_file_stream_reader_impl.h"
 #include "chrome/browser/chromeos/drive/fileapi/webkit_file_stream_writer_impl.h"
 #include "chrome/browser/drive/drive_api_util.h"
 #include "content/public/browser/browser_thread.h"
 #include "webkit/browser/blob/file_stream_reader.h"
 #include "webkit/browser/fileapi/async_file_util.h"
 #include "webkit/browser/fileapi/file_system_context.h"
 #include "webkit/browser/fileapi/file_system_url.h"
 
 using content::BrowserThread;
 
 namespace drive {
+namespace {
+
+void GetURLForBrowserTabOnUIThreadWithResourceEntry(

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

nit: Comments are missing. Especially we should comment that callback is called
on IO thread.

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

Done.

+    const FileSystemBackendDelegate::URLCallback& callback,
+    FileError error,
+    scoped_ptr<ResourceEntry> entry) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  GURL url;
+  if (error == FILE_ERROR_OK && entry->has_file_specific_info() &&
+      entry->file_specific_info().is_hosted_document()) {
+    url = GURL(entry->file_specific_info().alternate_url());
+  }
+  BrowserThread::PostTask(
+      BrowserThread::IO, FROM_HERE, base::Bind(callback, url));
+}
+
+void GetURLForBrowserTabOnUIThread(
+    const storage::FileSystemURL& url,
+    const FileSystemBackendDelegate::URLCallback& callback) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  FileSystemInterface* const file_system =
+      fileapi_internal::GetFileSystemFromUrl(url);
+  if (!file_system) {
+    BrowserThread::PostTask(
+        BrowserThread::IO, FROM_HERE, base::Bind(callback, GURL()));
+    return;
+  }
+  const base::FilePath file_path = util::ExtractDrivePathFromFileSystemUrl(url);
+  file_system->GetResourceEntry(
+      file_path,
+      base::Bind(&GetURLForBrowserTabOnUIThreadWithResourceEntry, callback));
+

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

nit: The empty line should be one line lower?

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

Done.

+}
+}  // namespace
 
 FileSystemBackendDelegate::FileSystemBackendDelegate()
     : async_file_util_(new internal::AsyncFileUtil) {
 }
 
 FileSystemBackendDelegate::~FileSystemBackendDelegate() {
 }
 
 storage::AsyncFileUtil* FileSystemBackendDelegate::GetAsyncFileUtil(
     storage::FileSystemType type) {
@@ skipping 44 matching lines @@
           file_path,
           offset));
 }
 
 storage::WatcherManager* FileSystemBackendDelegate::GetWatcherManager(
     const storage::FileSystemURL& url) {
   NOTIMPLEMENTED();
   return NULL;
 }
 
+void FileSystemBackendDelegate::GetURLForBrowserTab(
+    const storage::FileSystemURL& url,
+    const URLCallback& callback) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  BrowserThread::PostTask(
+      BrowserThread::UI,
+      FROM_HERE,
+      base::Bind(&GetURLForBrowserTabOnUIThread, url, callback));
+}
+
 }  // namespace drive