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

Side by Side Diff: chrome/browser/extensions/api/socket/tcp_socket_unittest.cc

Issue 11419224: Add missing (and remove superfluous) 'explicit' from constructors. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + remove non-straightforward changes Created 7 years, 11 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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "chrome/browser/extensions/api/socket/tcp_socket.h" 5 #include "chrome/browser/extensions/api/socket/tcp_socket.h"
6 6
7 #include "base/memory/scoped_ptr.h" 7 #include "base/memory/scoped_ptr.h"
8 #include "net/base/address_list.h" 8 #include "net/base/address_list.h"
9 #include "net/base/completion_callback.h" 9 #include "net/base/completion_callback.h"
10 #include "net/base/io_buffer.h" 10 #include "net/base/io_buffer.h"
(...skipping 25 matching lines...) Expand all
36 virtual bool IsConnected() const OVERRIDE { 36 virtual bool IsConnected() const OVERRIDE {
37 return true; 37 return true;
38 } 38 }
39 39
40 private: 40 private:
41 DISALLOW_COPY_AND_ASSIGN(MockTCPSocket); 41 DISALLOW_COPY_AND_ASSIGN(MockTCPSocket);
42 }; 42 };
43 43
44 class MockTCPServerSocket : public net::TCPServerSocket { 44 class MockTCPServerSocket : public net::TCPServerSocket {
45 public: 45 public:
46 explicit MockTCPServerSocket() 46 MockTCPServerSocket()
47 : net::TCPServerSocket(NULL, net::NetLog::Source()) { 47 : net::TCPServerSocket(NULL, net::NetLog::Source()) {
48 } 48 }
49 MOCK_METHOD2(Listen, int(const net::IPEndPoint& address, int backlog)); 49 MOCK_METHOD2(Listen, int(const net::IPEndPoint& address, int backlog));
50 MOCK_METHOD2(Accept, int(scoped_ptr<net::StreamSocket>* socket, 50 MOCK_METHOD2(Accept, int(scoped_ptr<net::StreamSocket>* socket,
51 const net::CompletionCallback& callback)); 51 const net::CompletionCallback& callback));
52 52
53 private: 53 private:
54 DISALLOW_COPY_AND_ASSIGN(MockTCPServerSocket); 54 DISALLOW_COPY_AND_ASSIGN(MockTCPServerSocket);
55 }; 55 };
56 56
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after
225 EXPECT_CALL(*tcp_server_socket, Listen(_, _)).Times(1); 225 EXPECT_CALL(*tcp_server_socket, Listen(_, _)).Times(1);
226 EXPECT_CALL(handler, OnAccept(_, _)); 226 EXPECT_CALL(handler, OnAccept(_, _));
227 227
228 std::string err_msg; 228 std::string err_msg;
229 EXPECT_EQ(net::OK, socket->Listen("127.0.0.1", 9999, 10, &err_msg)); 229 EXPECT_EQ(net::OK, socket->Listen("127.0.0.1", 9999, 10, &err_msg));
230 socket->Accept(base::Bind(&CompleteHandler::OnAccept, 230 socket->Accept(base::Bind(&CompleteHandler::OnAccept,
231 base::Unretained(&handler))); 231 base::Unretained(&handler)));
232 } 232 }
233 233
234 } // namespace extensions 234 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/extensions/api/serial/serial_apitest.cc ('k') | chrome/browser/extensions/extension_function_registry.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698