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

Side by Side Diff: remoting/webapp/base/js/log_to_server.js

Issue 1131853004: Include the session id in feedback logs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed blank line. Created 5 years, 7 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
« no previous file with comments | « remoting/webapp/base/js/client_session.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 * Module for sending log entries to the server. 7 * Module for sending log entries to the server.
8 */ 8 */
9 9
10 'use strict'; 10 'use strict';
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 */ 103 */
104 remoting.LogToServer.prototype.logSignalStrategyProgress = 104 remoting.LogToServer.prototype.logSignalStrategyProgress =
105 function(strategyType, progress) { 105 function(strategyType, progress) {
106 this.maybeExpireSessionId_(); 106 this.maybeExpireSessionId_();
107 var entry = remoting.ServerLogEntry.makeSignalStrategyProgress( 107 var entry = remoting.ServerLogEntry.makeSignalStrategyProgress(
108 this.sessionId_, strategyType, progress); 108 this.sessionId_, strategyType, progress);
109 this.log_(entry); 109 this.log_(entry);
110 }; 110 };
111 111
112 /** 112 /**
113 * @return {string} The current session id. This is random GUID, refreshed
114 * every 24hrs.
115 */
116 remoting.LogToServer.prototype.getSessionId = function() {
117 return this.sessionId_;
118 };
119
120 /**
113 * Whether a session state is one of the states that occurs at the start of 121 * Whether a session state is one of the states that occurs at the start of
114 * a session. 122 * a session.
115 * 123 *
116 * @private 124 * @private
117 * @param {remoting.ClientSession.State} state 125 * @param {remoting.ClientSession.State} state
118 * @return {boolean} 126 * @return {boolean}
119 */ 127 */
120 remoting.LogToServer.isStartOfSession_ = function(state) { 128 remoting.LogToServer.isStartOfSession_ = function(state) {
121 return ((state == remoting.ClientSession.State.CONNECTING) || 129 return ((state == remoting.ClientSession.State.CONNECTING) ||
122 (state == remoting.ClientSession.State.INITIALIZING) || 130 (state == remoting.ClientSession.State.INITIALIZING) ||
(...skipping 156 matching lines...) Expand 10 before | Expand all | Expand 10 after
279 this.authTotalTime_ = totalTime; 287 this.authTotalTime_ = totalTime;
280 }; 288 };
281 289
282 /** 290 /**
283 * @param {string} hostVersion Version of the host for current session. 291 * @param {string} hostVersion Version of the host for current session.
284 * @return {void} Nothing. 292 * @return {void} Nothing.
285 */ 293 */
286 remoting.LogToServer.prototype.setHostVersion = function(hostVersion) { 294 remoting.LogToServer.prototype.setHostVersion = function(hostVersion) {
287 this.hostVersion_ = hostVersion; 295 this.hostVersion_ = hostVersion;
288 }; 296 };
289
OLDNEW
« no previous file with comments | « remoting/webapp/base/js/client_session.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698