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

Side by Side Diff: remoting/webapp/browser_test/mock_session_connector.js

Issue 981083002: [Chromoting] Move ownership of ClientPlugin into SessionConnector. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove createPluginAndConnect Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 /** 5 /**
6 * @fileoverview 6 * @fileoverview
7 * Mock implementation of SessionConnector for testing. 7 * Mock implementation of SessionConnector for testing.
8 * @suppress {checkTypes} 8 * @suppress {checkTypes}
9 */ 9 */
10 10
(...skipping 131 matching lines...) Expand 10 before | Expand all | Expand 10 after
142 /** @param {remoting.ClientSession.StateEvent} event */ 142 /** @param {remoting.ClientSession.StateEvent} event */
143 var onStateChange = function(event) { 143 var onStateChange = function(event) {
144 if (event.current == remoting.ClientSession.State.CONNECTED) { 144 if (event.current == remoting.ClientSession.State.CONNECTED) {
145 that.onConnected_(clientSession); 145 that.onConnected_(clientSession);
146 } 146 }
147 }; 147 };
148 148
149 clientSession.addEventListener( 149 clientSession.addEventListener(
150 remoting.ClientSession.Events.stateChanged, 150 remoting.ClientSession.Events.stateChanged,
151 onStateChange); 151 onStateChange);
152 clientSession.createPluginAndConnect(this.onExtensionMessage_);
garykac 2015/03/06 00:45:04 This mock is not being used and will be removed so
153 }; 152 };
154 153
155 154
156 /** 155 /**
157 * @constructor 156 * @constructor
158 * @extends {remoting.SessionConnectorFactory} 157 * @extends {remoting.SessionConnectorFactory}
159 */ 158 */
160 remoting.MockSessionConnectorFactory = function() {}; 159 remoting.MockSessionConnectorFactory = function() {};
161 160
162 /** 161 /**
(...skipping 12 matching lines...) Expand all
175 * in the client session. 174 * in the client session.
176 * @return {remoting.MockSessionConnector} 175 * @return {remoting.MockSessionConnector}
177 */ 176 */
178 remoting.MockSessionConnectorFactory.prototype.createConnector = 177 remoting.MockSessionConnectorFactory.prototype.createConnector =
179 function(clientContainer, onConnected, onError, onExtensionMessage, 178 function(clientContainer, onConnected, onError, onExtensionMessage,
180 onConnectionFailed, requiredCapabilities, defaultRemapKeys) { 179 onConnectionFailed, requiredCapabilities, defaultRemapKeys) {
181 return new remoting.MockSessionConnector( 180 return new remoting.MockSessionConnector(
182 clientContainer, onConnected, onError, onExtensionMessage, 181 clientContainer, onConnected, onError, onExtensionMessage,
183 onConnectionFailed, requiredCapabilities, defaultRemapKeys); 182 onConnectionFailed, requiredCapabilities, defaultRemapKeys);
184 }; 183 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698