Fix IWYU in base/files/file.h

base/files/file.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 BASE_FILES_FILE_H_
 #define BASE_FILES_FILE_H_
 
 #include "build/build_config.h"
 #if defined(OS_WIN)
 #include <windows.h>
 #endif
 
-#include <string>
-
 #include "base/base_export.h"
 #include "base/basictypes.h"
-#include "base/files/file_path.h"
+//#include "base/files/file_path.h"

[Nico, 2014/03/07 15:30:25]

?

[Nico, 2014/03/07 15:32:01]

lgtm with this line removed instead of commented out

[Thiemo Nagel, 2014/03/07 16:07:38]

Thank you for catching this!  I had commented this out for testing and it was an
oversight to leave it there for the commit.

 #include "base/move.h"
 #include "base/time/time.h"
 
 #if defined(OS_WIN)
 #include "base/win/scoped_handle.h"
 #endif
 
 namespace base {
 
+class FilePath;
+
 #if defined(OS_WIN)
 typedef HANDLE PlatformFile;
 #elif defined(OS_POSIX)
 typedef int PlatformFile;
 #endif
 
 
 // Thin wrapper around an OS-level file.
 // Note that this class does not provide any support for asynchronous IO, other
 // than the ability to create asynchronous handles on Windows.
@@ skipping 244 matching lines @@
 #endif
 
   Error error_details_;
   bool created_;
   bool async_;
 };
 
 }  // namespace base
 
 #endif  // BASE_FILES_FILE_H_