Move fileapi into its own component

webkit/fileapi/file_system_operation_context.h

 // Copyright (c) 2012 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_FILEAPI_FILE_SYSTEM_OPERATION_CONTEXT_H_
 #define WEBKIT_FILEAPI_FILE_SYSTEM_OPERATION_CONTEXT_H_
 
 #include "base/logging.h"
 #include "base/memory/ref_counted.h"
 #include "googleurl/src/gurl.h"
+#include "webkit/fileapi/fileapi_export.h"
+#include "webkit/fileapi/file_system_context.h"

[kinuko, 2012/05/29 10:55:36]

Why we need this to be included?

[benwells, 2012/05/29 11:40:22]

On Windows this was needed to compile.

I assume adding the dll export for the FileSystemOperationContext meant it
needed to have full information about this type it is using. I'm not sure if
it's because of the getter or the scoped refptr.

[kinuko, 2012/05/29 11:47:54]

Ok thanks for checking!

 #include "webkit/fileapi/file_system_file_util.h"
 #include "webkit/fileapi/file_system_types.h"
 
 namespace base {
 class SequencedTaskRunner;
 }
 
 namespace fileapi {
 
-class FileSystemContext;
-
-class FileSystemOperationContext {
+class FILEAPI_EXPORT_PRIVATE FileSystemOperationContext {
  public:
   explicit FileSystemOperationContext(FileSystemContext* context);
   ~FileSystemOperationContext();
 
   FileSystemContext* file_system_context() const {
     return file_system_context_.get();
   }
 
   void set_allowed_bytes_growth(const int64& allowed_bytes_growth) {
     allowed_bytes_growth_ = allowed_bytes_growth;
   }
   int64 allowed_bytes_growth() const { return allowed_bytes_growth_; }
 
   base::SequencedTaskRunner* file_task_runner() const;
 
  private:
   scoped_refptr<FileSystemContext> file_system_context_;
 
   int64 allowed_bytes_growth_;
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_FILE_SYSTEM_OPERATION_CONTEXT_H_