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

Unified Diff: chrome/browser/extensions/api/cast_channel/cast_socket.h

Issue 393023003: Added connection timeout functionality to CastSocket. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added missing initializer for CastSocket unit tests. Created 6 years, 5 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/extensions/api/cast_channel/cast_socket.h
diff --git a/chrome/browser/extensions/api/cast_channel/cast_socket.h b/chrome/browser/extensions/api/cast_channel/cast_socket.h
index dba56d7196f75314ba0a6c59b4f4db566a787a12..677095506f2966bea7bc4bf826b4c6de8a1ea3ca 100644
--- a/chrome/browser/extensions/api/cast_channel/cast_socket.h
+++ b/chrome/browser/extensions/api/cast_channel/cast_socket.h
@@ -10,10 +10,12 @@
#include "base/basictypes.h"
#include "base/callback.h"
+#include "base/cancelable_callback.h"
#include "base/gtest_prod_util.h"
#include "base/memory/ref_counted.h"
#include "base/memory/weak_ptr.h"
#include "base/threading/thread_checker.h"
+#include "base/timer/timer.h"
#include "chrome/common/extensions/api/cast_channel.h"
#include "extensions/browser/api/api_resource.h"
#include "extensions/browser/api/api_resource_manager.h"
@@ -68,7 +70,8 @@ class CastSocket : public ApiResource,
const net::IPEndPoint& ip_endpoint,
ChannelAuthType channel_auth,
CastSocket::Delegate* delegate,
- net::NetLog* net_log);
+ net::NetLog* net_log,
+ const base::TimeDelta& connect_timeout);
virtual ~CastSocket();
// The IP endpoint for the destination of the channel.
@@ -144,6 +147,7 @@ class CastSocket : public ApiResource,
private:
friend class ApiResourceManager<CastSocket>;
friend class CastSocketTest;
+ friend class TestCastSocket;
static const char* service_name() { return "CastSocketManager"; }
@@ -191,6 +195,10 @@ class CastSocket : public ApiResource,
// 2. Certificate is rooted to a trusted CA.
virtual bool VerifyChallengeReply();
+ // Invoked by a cancelable closure when connection setup time
+ // exceeds the interval specified at |connect_timeout|.
+ void CancelConnect();
+
/////////////////////////////////////////////////////////////////////////////
// Following methods work together to implement the following flow:
// 1. Create a new TCP socket and connect to it
@@ -267,6 +275,8 @@ class CastSocket : public ApiResource,
virtual bool CalledOnValidThread() const;
+ virtual base::Timer* GetTimer();
+
base::ThreadChecker thread_checker_;
// The id of the channel.
@@ -311,9 +321,17 @@ class CastSocket : public ApiResource,
// Reply received from the receiver to a challenge request.
scoped_ptr<CastMessage> challenge_reply_;
- // Callback invoked when the socket is connected.
+ // Callback invoked when the socket is connected or fails to connect.
net::CompletionCallback connect_callback_;
+ // Duration to wait before timing out.
+ base::TimeDelta connect_timeout_;
+ // Timer invoked when the connection has timed out.
+ scoped_ptr<base::Timer> connect_timeout_timer_;
+ // Set when a timeout is triggered and the connection process has
+ // canceled.
+ bool is_canceled_;
+
// Connection flow state machine state.
ConnectionState connect_state_;
// Write flow state machine state.

Powered by Google App Engine
This is Rietveld 408576698