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

Unified Diff: remoting/protocol/client_stub.h

Issue 6724033: Remove authenticated_ fields from stubs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix .gyp file for unit tests. Created 9 years, 9 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: remoting/protocol/client_stub.h
diff --git a/remoting/protocol/client_stub.h b/remoting/protocol/client_stub.h
index d6550ec02c96d8bce3ab356d14eac0ed717b933d..4a75e05903bbb04e2f92701833cea4100f841a71 100644
--- a/remoting/protocol/client_stub.h
+++ b/remoting/protocol/client_stub.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -22,35 +22,15 @@ class NotifyResolutionRequest;
class ClientStub {
public:
- ClientStub();
- virtual ~ClientStub();
+ ClientStub() {}
+ virtual ~ClientStub() {}
virtual void NotifyResolution(const NotifyResolutionRequest* msg,
Task* done) = 0;
virtual void BeginSessionResponse(const LocalLoginStatus* msg,
Task* done) = 0;
- // TODO(lambroslambrou): Remove OnAuthenticated() and OnClosed() when stubs
- // are refactored not to store authentication state.
-
- // Called when the client has authenticated with the host to enable the
- // host->client control channel.
- // Before this is called, only a limited set of control messages will be
- // processed.
- void OnAuthenticated();
-
- // Called when the client is no longer connected.
- void OnClosed();
-
- // Has the client successfully authenticated with the host?
- // I.e., should we be processing control events?
- bool authenticated();
-
private:
- // Initially false, this records whether the client has authenticated with
- // the host.
- bool authenticated_;
-
DISALLOW_COPY_AND_ASSIGN(ClientStub);
};

Powered by Google App Engine
This is Rietveld 408576698