Add GetURLForBrowserTab method to the external file system backend.

chrome/browser/chromeos/fileapi/file_system_backend_delegate.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_FILEAPI_FILE_SYSTEM_BACKEND_DELEGATE_H_
 #define CHROME_BROWSER_CHROMEOS_FILEAPI_FILE_SYSTEM_BACKEND_DELEGATE_H_
 
 #include "base/basictypes.h"
+#include "base/callback_forward.h"
 #include "base/memory/scoped_ptr.h"
 #include "webkit/common/fileapi/file_system_types.h"
 
+class GURL;
+
 namespace base {
 class Time;
 }  // namespace base
 
 namespace storage {
 class AsyncFileUtil;
 class FileSystemContext;
 class FileStreamReader;
 class FileSystemURL;
 class FileStreamWriter;
 class WatcherManager;
 }  // namespace storage

[mtomasz, 2014/09/04 06:18:51]

nit: Not realted to this CL, but can be easily merged with #28?

[hirono, 2014/09/04 07:15:38]

Done.

 
 namespace storage {
 class FileStreamReader;
+typedef base::Callback<void(const GURL&)> URLCallback;

[mtomasz, 2014/09/04 06:18:51]

nit: This is already declared in webkit/browser/fileapi/file_system_backend.h.

Shall we add an include instead?

[hirono, 2014/09/04 07:15:38]

Done.

 }  // namespace storage
 
 namespace chromeos {
 
 // This is delegate interface to inject the implementation of the some methods
 // of FileSystemBackend.
 class FileSystemBackendDelegate {
  public:
   virtual ~FileSystemBackendDelegate() {}
 
@@ skipping 11 matching lines @@
   // Called from FileSystemBackend::CreateFileStreamWriter().
   virtual scoped_ptr<storage::FileStreamWriter> CreateFileStreamWriter(
       const storage::FileSystemURL& url,
       int64 offset,
       storage::FileSystemContext* context) = 0;
 
   // Called from the FileSystemWatcherService class. The returned pointer must
   // stay valid until shutdown.
   virtual storage::WatcherManager* GetWatcherManager(
       const storage::FileSystemURL& url) = 0;
+
+  // Called from FileSystemBackend::GetRedirectURLForBrowserTab.  Please ensure
+  // that the returned URL is secure. The URL is to be redirected to and opened
+  // in a browser tab.

[mtomasz, 2014/09/04 06:18:51]

nit: Actually not only browser tab. It can be also opened in <img>, <video> in
apps, extensions, as well as via XmlHttpRequest. Right?

If so, then maybe GetRedirectURLForContents? WDYT?

[hirono, 2014/09/04 07:15:38]

Done.

+  virtual void GetRedirectURLForBrowserTab(
+      const storage::FileSystemURL& url,
+      const storage::URLCallback& callback) = 0;
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_FILEAPI_FILE_SYSTEM_BACKEND_DELEGATE_H_