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 {

[kinaba, 2014/09/04 04:17:04]

put a blank line between namespace { the following declaration.

[hirono, 2014/09/04 04:29:09]

Done.

+void PostURLOnIOThread(const FileSystemBackendDelegate::URLCallback& callback,

[kinaba, 2014/09/04 04:17:04]

Maybe: PostURLToIOThread?

Or I think you can remove this function and in line the
BrowserThread::PostTask(BrowserThread::IO, ...) call, which is descriptive and
concise enough.

[hirono, 2014/09/04 04:29:09]

Removed the function.

+                       const GURL& url) {
+  BrowserThread::PostTask(
+      BrowserThread::IO, FROM_HERE, base::Bind(callback, url));
+}
+
+void GetURLForBrowserTabOnUIThreadWithResourceEntry(
+    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());
+  }
+  PostURLOnIOThread(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) {
+    PostURLOnIOThread(callback, GURL());
+    return;
+  }
+  const base::FilePath file_path = util::ExtractDrivePathFromFileSystemUrl(url);
+  file_system->GetResourceEntry(
+      file_path,
+      base::Bind(GetURLForBrowserTabOnUIThreadWithResourceEntry, callback));

[kinaba, 2014/09/04 04:17:04]

&GetURLFor...

[hirono, 2014/09/04 04:29:09]

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));

[kinaba, 2014/09/04 04:17:04]

&GetURLFor...

[hirono, 2014/09/04 04:29:09]

Done.

+}
+
 }  // namespace drive