mediaview: Implement ArcDocumentsProviderBackendDelegate.

chrome/browser/chromeos/arc/fileapi/arc_documents_provider_backend_delegate.cc

 // Copyright 2016 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/arc/fileapi/arc_documents_provider_backend_delegate.h"
 
 #include <utility>
 
 #include "base/logging.h"
+#include "base/memory/ptr_util.h"
+#include "chrome/browser/chromeos/arc/fileapi/arc_content_file_system_url_util.h"
+#include "chrome/browser/chromeos/arc/fileapi/arc_documents_provider_file_stream_reader.h"
+#include "chrome/browser/chromeos/arc/fileapi/arc_documents_provider_root.h"
 #include "content/public/browser/browser_thread.h"
 #include "storage/browser/fileapi/file_stream_reader.h"
 #include "storage/browser/fileapi/file_stream_writer.h"
 #include "url/gurl.h"
 
 using content::BrowserThread;
 
 namespace arc {
 
 ArcDocumentsProviderBackendDelegate::ArcDocumentsProviderBackendDelegate()
-    : async_file_util_(&roots_) {}
+    : async_file_util_(&roots_), weak_ptr_factory_(this) {}
 
 ArcDocumentsProviderBackendDelegate::~ArcDocumentsProviderBackendDelegate() {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 }
 
 storage::AsyncFileUtil* ArcDocumentsProviderBackendDelegate::GetAsyncFileUtil(
     storage::FileSystemType type) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   return &async_file_util_;
 }
 
 std::unique_ptr<storage::FileStreamReader>
 ArcDocumentsProviderBackendDelegate::CreateFileStreamReader(
     const storage::FileSystemURL& url,
     int64_t offset,
     int64_t max_bytes_to_read,
     const base::Time& expected_modification_time,
     storage::FileSystemContext* context) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
-  NOTIMPLEMENTED();  // TODO(crbug.com/671511): Implement this function.
-  return nullptr;
+
+  auto reader = base::MakeUnique<ArcDocumentsProviderFileStreamReader>(offset);
+
+  base::FilePath path;
+  ArcDocumentsProviderRoot* root = roots_.ParseAndLookup(url, &path);
+  if (!root) {
+    reader->SetContentUrl(GURL());
+    return std::move(reader);
+  }
+
+  root->ResolveToContentUrl(
+      path, base::Bind(&ArcDocumentsProviderFileStreamReader::SetContentUrl,
+                       reader->GetWeakPtr()));
+  return std::move(reader);
 }
 
 std::unique_ptr<storage::FileStreamWriter>
 ArcDocumentsProviderBackendDelegate::CreateFileStreamWriter(
     const storage::FileSystemURL& url,
     int64_t offset,
     storage::FileSystemContext* context) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   NOTREACHED();  // Read-only file system.
   return nullptr;
 }
 
 storage::WatcherManager* ArcDocumentsProviderBackendDelegate::GetWatcherManager(
     storage::FileSystemType type) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   NOTREACHED();  // Non-watchable file system.
   return nullptr;
 }
 
 void ArcDocumentsProviderBackendDelegate::GetRedirectURLForContents(

[hashimoto, 2016/12/22 06:20:01]

qq: When is this method used, and when is the FileStreamReader impl used?
Do we really need both?

[Shuhei Takahashi, 2017/01/05 08:45:40]

I realized this function is never called by chromeos::FileSystemBackend. Let's
make this function empty, and revisit if it turns out we need to implement it.

     const storage::FileSystemURL& url,
     const storage::URLCallback& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
-  NOTIMPLEMENTED();  // TODO(crbug.com/671511): Implement this function.
-  callback.Run(GURL());
+
+  base::FilePath path;
+  ArcDocumentsProviderRoot* root = roots_.ParseAndLookup(url, &path);
+  if (!root) {
+    callback.Run(GURL());
+    return;
+  }
+
+  root->ResolveToContentUrl(
+      path, base::Bind(&ArcDocumentsProviderBackendDelegate::
+                           GetRedirectURLForContentsWithContentUrl,
+                       weak_ptr_factory_.GetWeakPtr(), callback));
+}
+
+void ArcDocumentsProviderBackendDelegate::
+    GetRedirectURLForContentsWithContentUrl(

[hashimoto, 2016/12/22 06:20:01]

nit: "GetRedirectURLFor..." sounds like this method triggers some action while
this is actually a callback.
How about something like "OnRedirectURLReady" or
"ConvertArcUrlToExternalFileUrlAndRunCallback"?

[Shuhei Takahashi, 2017/01/05 08:45:40]

Removed the unused function.

+        const storage::URLCallback& callback,
+        const GURL& content_url) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  callback.Run(ArcUrlToExternalFileUrl(content_url));
 }
 
 }  // namespace arc