net: Reset UploadDataStream on HttpNetworkTransaction destruction

net/http/http_network_transaction.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 "net/http/http_network_transaction.h"
 
 #include <set>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 145 matching lines @@
       } else if (stream_->IsSpdyHttpStream()) {
         // Doesn't really matter for SpdyHttpStream. Just close it.
         stream_->Close(true /* not reusable */);
       } else {
         // Otherwise, we try to drain the response body.
         HttpStreamBase* stream = stream_.release();
         stream->Drain(session_.get());
       }
     }
   }
+
+  if (request_ && request_->upload_data_stream)
+    request_->upload_data_stream->Reset();  // Invalidate pending operations.

[mmenke, 2013/10/02 01:02:54]

Comment isn't accurate - pending operations will still be completed, for the
most part (Since they don't, and can't, check the state of the weak pointer on
other threads).  Maybe "Invalidate pending callbacks."?

[hashimoto, 2013/10/02 03:45:35]

Done.

 }
 
 int HttpNetworkTransaction::Start(const HttpRequestInfo* request_info,
                                   const CompletionCallback& callback,
                                   const BoundNetLog& net_log) {
   SIMPLE_STATS_COUNTER("HttpNetworkTransaction.Count");
 
   net_log_ = net_log;
   request_ = request_info;
   start_time_ = base::Time::Now();
@@ skipping 1303 matching lines @@
       description = base::StringPrintf("Unknown state 0x%08X (%u)", state,
                                        state);
       break;
   }
   return description;
 }
 
 #undef STATE_CASE
 
 }  // namespace net