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

Unified Diff: chrome/browser/devtools/device/self_device_provider.cc

Issue 449883002: DevTools: Removed refcounting from AndroidWebSocket (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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: chrome/browser/devtools/device/self_device_provider.cc
diff --git a/chrome/browser/devtools/device/self_device_provider.cc b/chrome/browser/devtools/device/self_device_provider.cc
index ee0f0433594bea890f4f729e9f696049d12c7499..f6930b56353e13aa88a3f664617468b883adc591 100644
--- a/chrome/browser/devtools/device/self_device_provider.cc
+++ b/chrome/browser/devtools/device/self_device_provider.cc
@@ -17,9 +17,9 @@ const char kSerial[] = "local";
static void RunSocketCallback(
const AndroidDeviceManager::SocketCallback& callback,
- net::StreamSocket* socket,
+ scoped_ptr<net::StreamSocket> socket,
int result) {
- callback.Run(result, socket);
+ callback.Run(result, socket.Pass());
}
} // namespace
@@ -61,7 +61,8 @@ void SelfAsDeviceProvider::OpenSocket(const std::string& serial,
base::StringToInt(socket_name, &port);
net::AddressList address_list =
net::AddressList::CreateFromIPAddress(ip_number, port);
- net::TCPClientSocket* socket = new net::TCPClientSocket(
- address_list, NULL, net::NetLog::Source());
- socket->Connect(base::Bind(&RunSocketCallback, callback, socket));
+ scoped_ptr<net::StreamSocket> socket(new net::TCPClientSocket(
+ address_list, NULL, net::NetLog::Source()));
+ socket->Connect(
+ base::Bind(&RunSocketCallback, callback, base::Passed(&socket)));
}

Powered by Google App Engine
This is Rietveld 408576698