URLRequestJob: change ReadRawData contract

net/http/http_transaction_test_util.cc

Index: net/http/http_transaction_test_util.cc
diff --git a/net/http/http_transaction_test_util.cc b/net/http/http_transaction_test_util.cc
index 6bd8230c46363c106a347badef77bfe6a7f8c7e1..13df9f6cdd120cf96b54d034e20aaec091c00364 100644
--- a/net/http/http_transaction_test_util.cc
+++ b/net/http/http_transaction_test_util.cc
@@ -211,6 +211,7 @@ void TestTransactionConsumer::DidFinish(int result) {
 }
 
 void TestTransactionConsumer::Read() {
+  CHECK(!done_reading_called_);

[xunjieli, 2015/11/23 15:37:56]

I can't add CHECK(!done_reading_called_) here, because done_reading_called is a
member of MockNetworkTransaction.

[mmenke, 2015/11/23 16:06:02]

Sorry, I meant in MockNetworkTransaction::Read.

[xunjieli, 2015/11/23 16:15:58]

Sorry, I meant to say it is a member of MockNetworkLayer. I have added a
done_reading_called_ to MockNetworkTransaction.

   state_ = READING;
   read_buf_ = new IOBuffer(1024);
   int result = trans_->Read(read_buf_.get(),
@@ -506,6 +507,7 @@ MockNetworkLayer::MockNetworkLayer()
 MockNetworkLayer::~MockNetworkLayer() {}
 
 void MockNetworkLayer::TransactionDoneReading() {
+  CHECK(!done_reading_called_);
   done_reading_called_ = true;
 }