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

Unified Diff: net/socket/socks5_client_socket_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
« no previous file with comments | « net/socket/socket_test_util.cc ('k') | net/socket/socks_client_socket_pool_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/socket/socks5_client_socket_unittest.cc
===================================================================
--- net/socket/socks5_client_socket_unittest.cc (revision 123143)
+++ net/socket/socks5_client_socket_unittest.cc (working copy)
@@ -115,13 +115,13 @@
};
MockWrite data_writes[] = {
- MockWrite(true, kSOCKS5GreetRequest, kSOCKS5GreetRequestLength),
- MockWrite(true, kOkRequest, arraysize(kOkRequest)),
- MockWrite(true, payload_write.data(), payload_write.size()) };
+ MockWrite(ASYNC, kSOCKS5GreetRequest, kSOCKS5GreetRequestLength),
+ MockWrite(ASYNC, kOkRequest, arraysize(kOkRequest)),
+ MockWrite(ASYNC, payload_write.data(), payload_write.size()) };
MockRead data_reads[] = {
- MockRead(true, kSOCKS5GreetResponse, kSOCKS5GreetResponseLength),
- MockRead(true, kSOCKS5OkResponse, kSOCKS5OkResponseLength),
- MockRead(true, payload_read.data(), payload_read.size()) };
+ MockRead(ASYNC, kSOCKS5GreetResponse, kSOCKS5GreetResponseLength),
+ MockRead(ASYNC, kSOCKS5OkResponse, kSOCKS5OkResponseLength),
+ MockRead(ASYNC, payload_read.data(), payload_read.size()) };
user_sock_.reset(BuildMockSocket(data_reads, arraysize(data_reads),
data_writes, arraysize(data_writes),
@@ -185,12 +185,12 @@
for (int i = 0; i < 2; ++i) {
MockWrite data_writes[] = {
- MockWrite(false, kSOCKS5GreetRequest, kSOCKS5GreetRequestLength),
- MockWrite(false, request.data(), request.size())
+ MockWrite(SYNCHRONOUS, kSOCKS5GreetRequest, kSOCKS5GreetRequestLength),
+ MockWrite(SYNCHRONOUS, request.data(), request.size())
};
MockRead data_reads[] = {
- MockRead(false, kSOCKS5GreetResponse, kSOCKS5GreetResponseLength),
- MockRead(false, kSOCKS5OkResponse, kSOCKS5OkResponseLength)
+ MockRead(SYNCHRONOUS, kSOCKS5GreetResponse, kSOCKS5GreetResponseLength),
+ MockRead(SYNCHRONOUS, kSOCKS5OkResponse, kSOCKS5OkResponseLength)
};
user_sock_.reset(BuildMockSocket(data_reads, arraysize(data_reads),
@@ -245,12 +245,12 @@
const char partial1[] = { 0x05, 0x01 };
const char partial2[] = { 0x00 };
MockWrite data_writes[] = {
- MockWrite(true, arraysize(partial1)),
- MockWrite(true, partial2, arraysize(partial2)),
- MockWrite(true, kOkRequest, arraysize(kOkRequest)) };
+ MockWrite(ASYNC, arraysize(partial1)),
+ MockWrite(ASYNC, partial2, arraysize(partial2)),
+ MockWrite(ASYNC, kOkRequest, arraysize(kOkRequest)) };
MockRead data_reads[] = {
- MockRead(true, kSOCKS5GreetResponse, kSOCKS5GreetResponseLength),
- MockRead(true, kSOCKS5OkResponse, kSOCKS5OkResponseLength) };
+ MockRead(ASYNC, kSOCKS5GreetResponse, kSOCKS5GreetResponseLength),
+ MockRead(ASYNC, kSOCKS5OkResponse, kSOCKS5OkResponseLength) };
user_sock_.reset(BuildMockSocket(data_reads, arraysize(data_reads),
data_writes, arraysize(data_writes),
hostname, 80, &net_log_));
@@ -276,12 +276,12 @@
const char partial1[] = { 0x05 };
const char partial2[] = { 0x00 };
MockWrite data_writes[] = {
- MockWrite(true, kSOCKS5GreetRequest, kSOCKS5GreetRequestLength),
- MockWrite(true, kOkRequest, arraysize(kOkRequest)) };
+ MockWrite(ASYNC, kSOCKS5GreetRequest, kSOCKS5GreetRequestLength),
+ MockWrite(ASYNC, kOkRequest, arraysize(kOkRequest)) };
MockRead data_reads[] = {
- MockRead(true, partial1, arraysize(partial1)),
- MockRead(true, partial2, arraysize(partial2)),
- MockRead(true, kSOCKS5OkResponse, kSOCKS5OkResponseLength) };
+ MockRead(ASYNC, partial1, arraysize(partial1)),
+ MockRead(ASYNC, partial2, arraysize(partial2)),
+ MockRead(ASYNC, kSOCKS5OkResponse, kSOCKS5OkResponseLength) };
user_sock_.reset(BuildMockSocket(data_reads, arraysize(data_reads),
data_writes, arraysize(data_writes),
hostname, 80, &net_log_));
@@ -304,14 +304,14 @@
{
const int kSplitPoint = 3; // Break handshake write into two parts.
MockWrite data_writes[] = {
- MockWrite(true, kSOCKS5GreetRequest, kSOCKS5GreetRequestLength),
- MockWrite(true, kOkRequest, kSplitPoint),
- MockWrite(true, kOkRequest + kSplitPoint,
+ MockWrite(ASYNC, kSOCKS5GreetRequest, kSOCKS5GreetRequestLength),
+ MockWrite(ASYNC, kOkRequest, kSplitPoint),
+ MockWrite(ASYNC, kOkRequest + kSplitPoint,
arraysize(kOkRequest) - kSplitPoint)
};
MockRead data_reads[] = {
- MockRead(true, kSOCKS5GreetResponse, kSOCKS5GreetResponseLength),
- MockRead(true, kSOCKS5OkResponse, kSOCKS5OkResponseLength) };
+ MockRead(ASYNC, kSOCKS5GreetResponse, kSOCKS5GreetResponseLength),
+ MockRead(ASYNC, kSOCKS5OkResponse, kSOCKS5OkResponseLength) };
user_sock_.reset(BuildMockSocket(data_reads, arraysize(data_reads),
data_writes, arraysize(data_writes),
hostname, 80, &net_log_));
@@ -333,13 +333,13 @@
{
const int kSplitPoint = 6; // Break the handshake read into two parts.
MockWrite data_writes[] = {
- MockWrite(true, kSOCKS5GreetRequest, kSOCKS5GreetRequestLength),
- MockWrite(true, kOkRequest, arraysize(kOkRequest))
+ MockWrite(ASYNC, kSOCKS5GreetRequest, kSOCKS5GreetRequestLength),
+ MockWrite(ASYNC, kOkRequest, arraysize(kOkRequest))
};
MockRead data_reads[] = {
- MockRead(true, kSOCKS5GreetResponse, kSOCKS5GreetResponseLength),
- MockRead(true, kSOCKS5OkResponse, kSplitPoint),
- MockRead(true, kSOCKS5OkResponse + kSplitPoint,
+ MockRead(ASYNC, kSOCKS5GreetResponse, kSOCKS5GreetResponseLength),
+ MockRead(ASYNC, kSOCKS5OkResponse, kSplitPoint),
+ MockRead(ASYNC, kSOCKS5OkResponse + kSplitPoint,
kSOCKS5OkResponseLength - kSplitPoint)
};
« no previous file with comments | « net/socket/socket_test_util.cc ('k') | net/socket/socks_client_socket_pool_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698