FileWriterDelegate should not call URLRequest::Start() after Cancel().

webkit/fileapi/file_writer_delegate.cc

Index: webkit/fileapi/file_writer_delegate.cc
diff --git a/webkit/fileapi/file_writer_delegate.cc b/webkit/fileapi/file_writer_delegate.cc
index d95e00be3ec1108307db8ec2204d79a80d355e31..0b741c1eef5bf1216120ac6a0775b7ff82479aaf 100644
--- a/webkit/fileapi/file_writer_delegate.cc
+++ b/webkit/fileapi/file_writer_delegate.cc
@@ -114,6 +114,10 @@ void FileWriterDelegate::OnGetFileInfoAndCallStartUpdate(
     OnError(error);
     return;
   }
+  if (request_->status().status() == net::URLRequestStatus::CANCELED) {

[ericu, 2012/04/09 16:14:22]

This looks more like "Don't call Start after Cancel" than "Don't call Cancel
before Start".  Is that what you wanted?  To prevent a call of Cancel before
Start I think you'd want to put code in OnError instead.

+    OnError(base::PLATFORM_FILE_ERROR_ABORT);
+    return;
+  }
   int64 allowed_bytes_growth =
       file_system_operation_context()->allowed_bytes_growth();
   if (allowed_bytes_growth < 0)