Revert of Add a parameter to FileStreamWriter::CreateForLocalFile to allow creating new

webkit/browser/fileapi/local_file_stream_writer.cc

 // 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.
 
 #include "webkit/browser/fileapi/local_file_stream_writer.h"
 
 #include "base/callback.h"
 #include "base/message_loop/message_loop.h"
 #include "net/base/file_stream.h"
 #include "net/base/io_buffer.h"
 #include "net/base/net_errors.h"
 
 namespace fileapi {
 
 namespace {
 
 const int kOpenFlagsForWrite = base::PLATFORM_FILE_OPEN |
                                base::PLATFORM_FILE_WRITE |
                                base::PLATFORM_FILE_ASYNC;
-const int kCreateFlagsForWrite = base::PLATFORM_FILE_CREATE |
-                                 base::PLATFORM_FILE_WRITE |
-                                 base::PLATFORM_FILE_ASYNC;
 
 }  // namespace
 
 FileStreamWriter* FileStreamWriter::CreateForLocalFile(
     base::TaskRunner* task_runner,
     const base::FilePath& file_path,
-    int64 initial_offset,
-    OpenOrCreate open_or_create) {
-  return new LocalFileStreamWriter(
-      task_runner, file_path, initial_offset, open_or_create);
+    int64 initial_offset) {
+  return new LocalFileStreamWriter(task_runner, file_path, initial_offset);
 }
 
 LocalFileStreamWriter::~LocalFileStreamWriter() {
   // Invalidate weak pointers so that we won't receive any callbacks from
   // in-flight stream operations, which might be triggered during the file close
   // in the FileStream destructor.
   weak_factory_.InvalidateWeakPtrs();
 
   // FileStream's destructor closes the file safely, since we opened the file
   // by its Open() method.
@@ skipping 36 matching lines @@
 
   has_pending_operation_ = true;
   int result = InitiateFlush(callback);
   if (result != net::ERR_IO_PENDING)
     has_pending_operation_ = false;
   return result;
 }
 
 LocalFileStreamWriter::LocalFileStreamWriter(base::TaskRunner* task_runner,
                                              const base::FilePath& file_path,
-                                             int64 initial_offset,
-                                             OpenOrCreate open_or_create)
+                                             int64 initial_offset)
     : file_path_(file_path),
-      open_or_create_(open_or_create),
       initial_offset_(initial_offset),
       task_runner_(task_runner),
       has_pending_operation_(false),
       weak_factory_(this) {}
 
 int LocalFileStreamWriter::InitiateOpen(
     const net::CompletionCallback& error_callback,
     const base::Closure& main_operation) {
   DCHECK(has_pending_operation_);
   DCHECK(!stream_impl_.get());
 
   stream_impl_.reset(new net::FileStream(NULL, task_runner_));
 
-  int open_flags = 0;
-  switch (open_or_create_) {
-  case OPEN_EXISTING_FILE:
-    open_flags = kOpenFlagsForWrite;
-    break;
-  case CREATE_NEW_FILE:
-    open_flags = kCreateFlagsForWrite;
-    break;
-  }
-
   return stream_impl_->Open(file_path_,
-                            open_flags,
+                            kOpenFlagsForWrite,
                             base::Bind(&LocalFileStreamWriter::DidOpen,
                                        weak_factory_.GetWeakPtr(),
                                        error_callback,
                                        main_operation));
 }
 
 void LocalFileStreamWriter::DidOpen(
     const net::CompletionCallback& error_callback,
     const base::Closure& main_operation,
     int result) {
@@ skipping 120 matching lines @@
     return false;
 
   net::CompletionCallback pending_cancel = cancel_callback_;
   has_pending_operation_ = false;
   cancel_callback_.Reset();
   pending_cancel.Run(net::OK);
   return true;
 }
 
 }  // namespace fileapi