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

Side by Side Diff: remoting/webapp/me2mom/client_session.js

Issue 8897017: Fix null pointer error. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Review Created 9 years 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 | Annotate | Revision Log
« no previous file with comments | « remoting/webapp/me2mom/client_screen.js ('k') | remoting/webapp/me2mom/toolbar.js » ('j') | 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 * Class handling creation and teardown of a remoting client session. 7 * Class handling creation and teardown of a remoting client session.
8 * 8 *
9 * This abstracts a <embed> element and controls the plugin which does the 9 * This abstracts a <embed> element and controls the plugin which does the
10 * actual remoting work. There should be no UI code inside this class. It 10 * actual remoting work. There should be no UI code inside this class. It
(...skipping 327 matching lines...) Expand 10 before | Expand all | Expand 10 after
338 this.onStateChange(oldState, newState); 338 this.onStateChange(oldState, newState);
339 } 339 }
340 this.logToServer.logClientSessionStateChange(this.state, this.error); 340 this.logToServer.logClientSessionStateChange(this.state, this.error);
341 }; 341 };
342 342
343 /** 343 /**
344 * This is a callback that gets called when the window is resized. 344 * This is a callback that gets called when the window is resized.
345 * 345 *
346 * @return {void} Nothing. 346 * @return {void} Nothing.
347 */ 347 */
348 remoting.ClientSession.prototype.onWindowSizeChanged = function() { 348 remoting.ClientSession.prototype.onResize = function() {
349 this.updateDimensions(); 349 this.updateDimensions();
350 }; 350 };
351 351
352 /** 352 /**
353 * This is a callback that gets called when the plugin notifies us of a change 353 * This is a callback that gets called when the plugin notifies us of a change
354 * in the size of the remote desktop. 354 * in the size of the remote desktop.
355 * 355 *
356 * @private 356 * @private
357 * @return {void} Nothing. 357 * @return {void} Nothing.
358 */ 358 */
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
416 return { 416 return {
417 'video_bandwidth': this.plugin.videoBandwidth, 417 'video_bandwidth': this.plugin.videoBandwidth,
418 'video_frame_rate': this.plugin.videoFrameRate, 418 'video_frame_rate': this.plugin.videoFrameRate,
419 'capture_latency': this.plugin.videoCaptureLatency, 419 'capture_latency': this.plugin.videoCaptureLatency,
420 'encode_latency': this.plugin.videoEncodeLatency, 420 'encode_latency': this.plugin.videoEncodeLatency,
421 'decode_latency': this.plugin.videoDecodeLatency, 421 'decode_latency': this.plugin.videoDecodeLatency,
422 'render_latency': this.plugin.videoRenderLatency, 422 'render_latency': this.plugin.videoRenderLatency,
423 'roundtrip_latency': this.plugin.roundTripLatency 423 'roundtrip_latency': this.plugin.roundTripLatency
424 }; 424 };
425 }; 425 };
OLDNEW
« no previous file with comments | « remoting/webapp/me2mom/client_screen.js ('k') | remoting/webapp/me2mom/toolbar.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698