Removes get upload progress plumbing. URLRequest queries the UploadDataStream directly.

net/base/upload_data_stream.h

Index: net/base/upload_data_stream.h
diff --git a/net/base/upload_data_stream.h b/net/base/upload_data_stream.h
index 9c269613e6952436c8cbd5ded1f309830c5816ab..88ddfe6637430502ecebf73689f5a42f41f6d053 100644
--- a/net/base/upload_data_stream.h
+++ b/net/base/upload_data_stream.h
@@ -13,6 +13,7 @@
 #include "base/macros.h"
 #include "net/base/completion_callback.h"
 #include "net/base/net_export.h"
+#include "net/base/upload_progress.h"
 #include "net/log/net_log.h"
 
 namespace net {
@@ -90,6 +91,10 @@ class NET_EXPORT UploadDataStream {
   virtual const std::vector<std::unique_ptr<UploadElementReader>>*
   GetElementReaders() const;
 
+  // Returns the upload progress. If the stream is not initialized
+  // succesfully, it returns empty UploadProgress.

[mmenke, 2016/09/12 19:35:31]

Maybe replace the second sentence with "If the stream was not initialized
successfully, or has been reset and not yet re-initialized, returns an empty
UploadProgress."

[mmenke, 2016/09/12 19:35:31]

Rather unexciting, but we should probably have a test for the new behaviors -
check value after Initialized() called, when it returns ERR_IO_PENDING, and
check value after Reset() is called.  May even want to do both twice, in a loop
(With a successful initialization and a check that it's true in between)

[shivanisha, 2016/09/14 20:40:25]

Updated the comment.
Added tests in http_stream_parser_unittest.cc

+  virtual UploadProgress GetUploadProgress() const;
+
  protected:
   // Must be called by subclasses when InitInternal and ReadInternal complete
   // asynchronously.