[Webapp Refactor] Cleans up the ClientSession.EventHandler interface.

remoting/webapp/crd/js/client_session_unittest.js

 // Copyright 2015 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.
 
 (function() {
 
 'use strict';
 
 /** @type {remoting.MockConnection} */
 var mockConnection;
 /** @type {remoting.ClientSession} */
 var session;
 /** @type {remoting.ClientSession.EventHandler} */
 var listener;
 /** @type {sinon.TestStub} */
 var logToServerStub;
 
 /**
  * @constructor
  * @implements {remoting.ClientSession.EventHandler}
  */
 var SessionListener = function() {};
 SessionListener.prototype.onConnectionFailed = function(error) {};
 SessionListener.prototype.onConnected = function(connectionInfo) {};
-SessionListener.prototype.onDisconnected = function() {};
-SessionListener.prototype.onError = function(error) {};
+SessionListener.prototype.onDisconnected = function(error) {};
 
 /**
  * @param {remoting.ClientSession.ConnectionError=} opt_error
  * @return {Promise}
  */
 function connect(opt_error) {
   var deferred = new base.Deferred();
   var host = new remoting.Host('fake_hostId');
   host.jabberId = 'fake_jid';
 
@@ skipping 98 matching lines @@
   });
 });
 
 QUnit.test(
   'Connection error after CONNECTED should raise the CONNECTION_DROPPED event',
   function(assert) {
 
   var State = remoting.ClientSession.State;
 
   return connect().then(function() {
-    var onError = sinon.stub(listener, 'onError');
+    var onDisconnected = sinon.stub(listener, 'onDisconnected');
     session.disconnect(new remoting.Error(remoting.Error.Tag.P2P_FAILURE));
-    assert.equal(onError.callCount, 1);
+    assert.equal(onDisconnected.callCount, 1);
     assert.equal(logToServerStub.args[2][0], State.CONNECTION_DROPPED);
   });
 });
 
 QUnit.test(
   'Connection error before CONNECTED should raise the CONNECTION_FAILED event',
   function(assert) {
 
   session.logHostOfflineErrors(true);
 
@@ skipping 10 matching lines @@
   });
 });
 
 QUnit.test('dispose() should dispose the plugin', function(assert) {
   var pluginDispose = sinon.stub(mockConnection.plugin(), 'dispose');
   session.dispose();
   assert.equal(pluginDispose.callCount, 1);
 });
 
 })();