SyncFS: Support resolveLocalFileSystemURL on SyncFileSystem

webkit/common/fileapi/file_system_info.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 WEBKIT_COMMON_FILEAPI_FILE_SYSTEM_INFO_H_
+#define WEBKIT_COMMON_FILEAPI_FILE_SYSTEM_INFO_H_
+
+#include "url/gurl.h"
+#include "webkit/common/fileapi/file_system_types.h"
+#include "webkit/common/webkit_storage_common_export.h"
+
+namespace fileapi {
+

[kinuko, 2013/09/09 12:52:23]

Can we add a struct comment?

"This struct is used to send the necessary information for Blink to create a
DOMFileSystem.  Since Blink side only uses
mount_type (rather than detailed/cracked filesystem type)
this only contains mount_type but not type."

[nhiroki, 2013/09/10 06:30:39]

Done.

+struct WEBKIT_STORAGE_COMMON_EXPORT FileSystemInfo {
+  FileSystemInfo();
+  FileSystemInfo(const std::string& filesystem_name,
+                 const GURL& root_url,
+                 fileapi::FileSystemType mount_type);
+
+  std::string filesystem_name;

[kinuko, 2013/09/09 12:52:23]

nit: Maybe just 'name' is ok here (since we're in a structure named
FileSystemInfo)

[nhiroki, 2013/09/10 06:30:39]

Done.

+  GURL root_url;
+  fileapi::FileSystemType mount_type;
+};
+
+}

[kinuko, 2013/09/09 12:52:23]

}  // namespace fileapi

[nhiroki, 2013/09/10 06:30:39]

Done.

+
+#endif  // WEBKIT_COMMON_FILEAPI_FILE_SYSTEM_INFO_H_