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

Unified Diff: remoting/webapp/crd/js/desktop_connected_view.js

Issue 2369013008: Log host and client screen sizes. (Closed)
Patch Set: Add DPI. Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/webapp/base/js/session_logger.js ('k') | remoting/webapp/crd/js/desktop_remoting_activity.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/crd/js/desktop_connected_view.js
diff --git a/remoting/webapp/crd/js/desktop_connected_view.js b/remoting/webapp/crd/js/desktop_connected_view.js
index 5696b7312f342cb4ea37646aa4c8113d386af95e..d67328971b7ac4be46625ff8868ca45751085ff9 100644
--- a/remoting/webapp/crd/js/desktop_connected_view.js
+++ b/remoting/webapp/crd/js/desktop_connected_view.js
@@ -15,11 +15,12 @@ var remoting = remoting || {};
/**
* @param {HTMLElement} container
* @param {remoting.ConnectionInfo} connectionInfo
+ * @param {remoting.SessionLogger} logger
* @constructor
* @extends {base.EventSourceImpl}
* @implements {base.Disposable}
*/
-remoting.DesktopConnectedView = function(container, connectionInfo) {
+remoting.DesktopConnectedView = function(container, connectionInfo, logger) {
/** @private {HTMLElement} */
this.container_ = container;
@@ -33,6 +34,9 @@ remoting.DesktopConnectedView = function(container, connectionInfo) {
/** @private */
this.host_ = connectionInfo.host();
+ /** @private */
+ this.logger_ = logger;
+
/** @private {remoting.DesktopViewport} */
this.viewport_ = null;
@@ -179,7 +183,8 @@ remoting.DesktopConnectedView.prototype.initUI_ = function() {
this.viewport_ = new remoting.DesktopViewport(
scrollerElement || document.body,
this.plugin_.hostDesktop(),
- this.host_.options);
+ this.host_.options,
+ this.logger_);
if (remoting.windowFrame) {
remoting.windowFrame.setDesktopConnectedView(this);
@@ -300,8 +305,10 @@ remoting.DesktopConnectedView.prototype.setRemapKeys = function(remappings) {
*
* @param {HTMLElement} container
* @param {remoting.ConnectionInfo} connectionInfo
+ * @param {remoting.SessionLogger} logger
* @return {remoting.DesktopConnectedView}
*/
-remoting.DesktopConnectedView.create = function(container, connectionInfo) {
- return new remoting.DesktopConnectedView(container, connectionInfo);
+remoting.DesktopConnectedView.create = function(container, connectionInfo,
+ logger) {
+ return new remoting.DesktopConnectedView(container, connectionInfo, logger);
};
« no previous file with comments | « remoting/webapp/base/js/session_logger.js ('k') | remoting/webapp/crd/js/desktop_remoting_activity.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698