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

Side by Side Diff: net/base/tcp_pinger_unittest.cc

Issue 118100: Avoid doing concurrent DNS resolves of the same hostname (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Get compiling on mac Created 11 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2006-2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/ref_counted.h" 5 #include "base/ref_counted.h"
6 #include "base/trace_event.h" 6 #include "base/trace_event.h"
7 #include "net/base/address_list.h" 7 #include "net/base/address_list.h"
8 #include "net/base/host_resolver.h" 8 #include "net/base/host_resolver.h"
9 #include "net/base/listen_socket.h" 9 #include "net/base/listen_socket.h"
10 #include "net/base/net_errors.h" 10 #include "net/base/net_errors.h"
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
60 listen_sock_ = ListenSocket::Listen("127.0.0.1", listen_port_, this); 60 listen_sock_ = ListenSocket::Listen("127.0.0.1", listen_port_, this);
61 if (listen_sock_.get()) break; 61 if (listen_sock_.get()) break;
62 } 62 }
63 ASSERT_TRUE(listen_sock_.get() != NULL); 63 ASSERT_TRUE(listen_sock_.get() != NULL);
64 } 64 }
65 65
66 TEST_F(TCPPingerTest, Ping) { 66 TEST_F(TCPPingerTest, Ping) {
67 net::AddressList addr; 67 net::AddressList addr;
68 net::HostResolver resolver; 68 net::HostResolver resolver;
69 69
70 int rv = resolver.Resolve("localhost", listen_port_, &addr, NULL); 70 int rv = resolver.Resolve("localhost", listen_port_, &addr, NULL, NULL);
71 EXPECT_EQ(rv, net::OK); 71 EXPECT_EQ(rv, net::OK);
72 72
73 net::TCPPinger pinger(addr); 73 net::TCPPinger pinger(addr);
74 rv = pinger.Ping(); 74 rv = pinger.Ping();
75 EXPECT_EQ(rv, net::OK); 75 EXPECT_EQ(rv, net::OK);
76 } 76 }
77 77
78 TEST_F(TCPPingerTest, PingFail) { 78 TEST_F(TCPPingerTest, PingFail) {
79 net::AddressList addr; 79 net::AddressList addr;
80 net::HostResolver resolver; 80 net::HostResolver resolver;
81 81
82 // "Kill" "server" 82 // "Kill" "server"
83 listen_sock_ = NULL; 83 listen_sock_ = NULL;
84 84
85 int rv = resolver.Resolve("localhost", listen_port_, &addr, NULL); 85 int rv = resolver.Resolve("localhost", listen_port_, &addr, NULL, NULL);
86 EXPECT_EQ(rv, net::OK); 86 EXPECT_EQ(rv, net::OK);
87 87
88 net::TCPPinger pinger(addr); 88 net::TCPPinger pinger(addr);
89 rv = pinger.Ping(base::TimeDelta::FromMilliseconds(100), 1); 89 rv = pinger.Ping(base::TimeDelta::FromMilliseconds(100), 1);
90 EXPECT_NE(rv, net::OK); 90 EXPECT_NE(rv, net::OK);
91 } 91 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698