[Chromoting] SessionLogger refactor.

remoting/webapp/base/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;
@@ skipping 58 matching lines @@
 }
 
 QUnit.module('ClientSession', {
   beforeEach: function() {
     chromeMocks.identity.mock$setToken('fake_token');
 
     mockConnection = new remoting.MockConnection();
     listener = new SessionListener();
     logger = new remoting.SessionLogger(remoting.ChromotingEvent.Role.CLIENT,
                                         base.doNothing);
-    logToServerStub = sinon.stub(logger, 'logClientSessionStateChange');
+    logToServerStub = sinon.stub(logger, 'logSessionStateChange');
   },
   afterEach: function() {
     session.dispose();
     mockConnection.restore();
   }
 });
 
 QUnit.test('should raise CONNECTED event on connected', function(assert) {
   return connect().then(function(){
     assert.ok(true, 'Expect session to connect.');
@@ skipping 24 matching lines @@
     var onDisconnected = sinon.stub(listener, 'onDisconnected');
     session.disconnect(remoting.Error.none());
     assert.equal(onDisconnected.callCount, 1);
   });
 });
 
 QUnit.test(
   'Connection error after CONNECTED should raise the CONNECTION_DROPPED event',
   function(assert) {
 
-  var State = remoting.ClientSession.State;
+  var State = remoting.ChromotingEvent.SessionState;
 
   return connect().then(function() {
     var onDisconnected = sinon.stub(listener, 'onDisconnected');
     session.disconnect(new remoting.Error(remoting.Error.Tag.P2P_FAILURE));
     assert.equal(onDisconnected.callCount, 1);
-    assert.equal(logToServerStub.args[2][0], State.CONNECTION_DROPPED);
+    assert.equal(logToServerStub.args[4][0], State.CONNECTION_DROPPED);
   });
 });
 
 QUnit.test(
   'Connection error before CONNECTED should raise the CONNECTION_FAILED event',
   function(assert) {
 
   var PluginError = remoting.ClientSession.ConnectionError;
-  var State = remoting.ClientSession.State;
+  var State = remoting.ChromotingEvent.SessionState;
 
   return connect(PluginError.SESSION_REJECTED).then(function() {
     assert.ok(false, 'Expect connection to fail');
   }).catch(function(/** remoting.Error */ error) {
     assert.ok(error.hasTag(remoting.Error.Tag.INVALID_ACCESS_CODE));
-    assert.equal(logToServerStub.args[1][0], State.FAILED);
-    var errorLogged = /** @type {remoting.Error} */(logToServerStub.args[1][1]);
+    assert.equal(logToServerStub.args[3][0], State.CONNECTION_FAILED);
+    var errorLogged = /** @type {remoting.Error} */(logToServerStub.args[3][1]);
     assert.equal(errorLogged.getTag(), remoting.Error.Tag.INVALID_ACCESS_CODE);
   });
 });
 
 })();