Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(128)

Unified Diff: net/dns/dns_transaction_unittest.cc

Issue 9425016: Change MockRead and MockWrite (et. al.) to take an IoMode enum, instead (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: sync Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/dns/dns_transaction_unittest.cc
===================================================================
--- net/dns/dns_transaction_unittest.cc (revision 123143)
+++ net/dns/dns_transaction_unittest.cc (working copy)
@@ -242,10 +242,10 @@
DnsResponse* response = new DnsResponse(data, data_length, 0);
responses_.push_back(response);
- writes_.push_back(MockWrite(true,
+ writes_.push_back(MockWrite(ASYNC,
query->io_buffer()->data(),
query->io_buffer()->size()));
- reads_.push_back(MockRead(true,
+ reads_.push_back(MockRead(ASYNC,
response->io_buffer()->data(),
data_length));
@@ -259,7 +259,7 @@
DnsQuery* query = new DnsQuery(id, DomainFromDot(dotted_name), qtype);
queries_.push_back(query);
- writes_.push_back(MockWrite(true,
+ writes_.push_back(MockWrite(ASYNC,
query->io_buffer()->data(),
query->io_buffer()->size()));
// Empty MockRead indicates no data.
@@ -284,10 +284,10 @@
header->flags |= htons(dns_protocol::kFlagResponse | rcode);
responses_.push_back(response);
- writes_.push_back(MockWrite(true,
+ writes_.push_back(MockWrite(ASYNC,
query->io_buffer()->data(),
query->io_buffer()->size()));
- reads_.push_back(MockRead(true,
+ reads_.push_back(MockRead(ASYNC,
response->io_buffer()->data(),
query->io_buffer()->size()));
transaction_ids_.push_back(id);
« no previous file with comments | « jingle/notifier/base/fake_ssl_client_socket_unittest.cc ('k') | net/ftp/ftp_network_transaction_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698