Add SetUploadDataStream method to URLFetcher.

net/url_request/test_url_fetcher_factory.cc

Index: net/url_request/test_url_fetcher_factory.cc
diff --git a/net/url_request/test_url_fetcher_factory.cc b/net/url_request/test_url_fetcher_factory.cc
index 08ba483b23a01d29811999120a0c365104a9f561..e645329f7f99c112a3430f7286f279b8caa6b3ff 100644
--- a/net/url_request/test_url_fetcher_factory.cc
+++ b/net/url_request/test_url_fetcher_factory.cc
@@ -11,6 +11,7 @@
 #include "base/memory/weak_ptr.h"
 #include "base/message_loop.h"
 #include "net/base/host_port_pair.h"
+#include "net/base/upload_data_stream.h"
 #include "net/http/http_response_headers.h"
 #include "net/url_request/url_fetcher_delegate.h"
 #include "net/url_request/url_fetcher_impl.h"
@@ -52,6 +53,12 @@ TestURLFetcher::~TestURLFetcher() {
     owner_->RemoveFetcherFromMap(id_);
 }
 
+void TestURLFetcher::SetUploadDataStream(
+    const std::string& upload_content_type,
+    scoped_ptr<UploadDataStream> upload_content) {
+  upload_data_stream_ = upload_content.Pass();
+}
+
 void TestURLFetcher::SetUploadData(const std::string& upload_content_type,
                                    const std::string& upload_content) {
   upload_data_ = upload_content;

[hashimoto, 2013/01/11 06:07:38]

Having both upload_data_stream_ and upload_data_ seems confusing.
How about completely replacing upload_data_ with upload_data_stream_, or mark
SetUploadDataStream as NOTIMPLEMENTED?

[mattm, 2013/01/11 21:42:02]

Thought about that, but wasn't entirely sure what the desired use case of this
class is.  If the test code actually wants to be able to distinguish whether the
caller used SetUploadData or SetUploadDataStream, then having them separately
would make sense.  Also this would make implementing the upload_data() accessor
harder.

[hashimoto, 2013/01/15 06:59:28]

I see.  Seems all users of this method just want to verify the correct value is
set as the upload data.
(http://code.google.com/p/chromium/source/search?q=\bupload_data\()