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

Unified Diff: remoting/webapp/base/js/chromoting_event.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 | « no previous file | remoting/webapp/base/js/session_logger.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/base/js/chromoting_event.js
diff --git a/remoting/webapp/base/js/chromoting_event.js b/remoting/webapp/base/js/chromoting_event.js
index c74dc4d4fdf9a56f4449f404e9895b2a16d6b457..2f9778803418f8b36aebeda8b7a2cbdcc0cf4325 100644
--- a/remoting/webapp/base/js/chromoting_event.js
+++ b/remoting/webapp/base/js/chromoting_event.js
@@ -120,6 +120,15 @@ remoting.ChromotingEvent = function(type) {
* @type {remoting.ChromotingEvent.FeatureTracker}
*/
this.feature_tracker;
+ /** @type {remoting.ChromotingEvent.ScreenResolution} */
+ this.host_all_screens_size;
+ /** @type {remoting.ChromotingEvent.ScreenResolution} */
+ this.client_video_size;
+ /** @type {remoting.ChromotingEvent.ScreenResolution} */
+ this.client_window_size;
+ /** @type {boolean} */
+ this.client_fullscreen;
+
this.init_();
};
@@ -230,6 +239,24 @@ remoting.ChromotingEvent.FeatureTracker = function() {
this.fullscreen_esc_count = 0;
};
+/**
+ * Client or host screen resolution.
+ *
+ * @param {number} width
+ * @param {number} height
+ * @param {number} dpi
+ * @struct
+ * @constructor
+ */
+ remoting.ChromotingEvent.ScreenResolution = function(width, height, dpi) {
+ /** @type {number} */
+ this.width = width;
+ /** @type {number} */
+ this.height = height;
+ /** @type {number} */
+ this.dpi = dpi;
+};
+
})();
/**
@@ -245,7 +272,8 @@ remoting.ChromotingEvent.Type = {
RESTART: 7,
HOST_STATUS: 8,
SIGNAL_STRATEGY_PROGRESS: 9,
- FEATURE_TRACKING: 10
+ FEATURE_TRACKING: 10,
+ SCREEN_RESOLUTIONS: 11,
};
/** @enum {number} */
« no previous file with comments | « no previous file | remoting/webapp/base/js/session_logger.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698