[Webapp Refactor] Reparent the ConnectedView into the delegate.

remoting/webapp/crd/js/desktop_connected_view.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.
 
 /**
  * @fileoverview
  * Class handling user-facing aspects of the client session.
  */
 
 'use strict';
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 /**
- * @param {remoting.ClientPlugin} plugin
  * @param {HTMLElement} container
- * @param {remoting.Host} host
- * @param {remoting.DesktopConnectedView.Mode} mode The mode of this connection.
+ * @param {remoting.ConnectionInfo} connectionInfo
  * @param {string} defaultRemapKeys The default set of remap keys, to use
  *     when the client doesn't define any.
  * @constructor
  * @extends {base.EventSourceImpl}
  * @implements {base.Disposable}
  */
-remoting.DesktopConnectedView = function(plugin, container, host, mode,
+remoting.DesktopConnectedView = function(container, connectionInfo,
                                          defaultRemapKeys) {
 
   /** @private {HTMLElement} */
   this.container_ = container;
 
   /** @private {remoting.ClientPlugin} */
-  this.plugin_ = plugin;
+  this.plugin_ = connectionInfo.plugin();
+
+  /** @private {remoting.ClientSession} */
+  this.session_ = connectionInfo.session();
 
   /** @private */
-  this.host_ = host;
+  this.host_ = connectionInfo.host();
 
   /** @private */
-  this.mode_ = mode;
+  this.mode_ = connectionInfo.mode();
 
   /** @private {string} */
   this.defaultRemapKeys_ = defaultRemapKeys;
 
-  /** @private {Element} */
-  this.debugRegionContainer_ =
-      this.container_.querySelector('.debug-region-container');
-
   /** @private {remoting.DesktopViewport} */
   this.viewport_ = null;
 
   /** private {remoting.ConnectedView} */
   this.view_ = null;
 
   /** @private {remoting.VideoFrameRecorder} */
   this.videoFrameRecorder_ = null;
 
   /** private {base.Disposable} */
   this.eventHooks_ = null;
 
-  this.setupPlugin_();
+  this.initPlugin_();
+  this.initUI_();
 };
 
 /** @return {void} Nothing. */
 remoting.DesktopConnectedView.prototype.dispose = function() {
   if (remoting.windowFrame) {
     remoting.windowFrame.setDesktopConnectedView(null);
   }
   if (remoting.toolbar) {
     remoting.toolbar.setDesktopConnectedView(null);
   }
@@ skipping 67 matching lines @@
 remoting.DesktopConnectedView.prototype.getPluginContainer_ = function() {
   return this.container_.querySelector('.client-plugin-container');
 };
 
 /** @return {remoting.DesktopViewport} */
 remoting.DesktopConnectedView.prototype.getViewportForTesting = function() {
   return this.viewport_;
 };
 
 /** @private */
-remoting.DesktopConnectedView.prototype.setupPlugin_ = function() {
+remoting.DesktopConnectedView.prototype.initPlugin_ = function() {
   // Show the Send Keys menu only if the plugin has the injectKeyEvent feature,
   // and the Ctrl-Alt-Del button only in Me2Me mode.
   if (!this.plugin_.hasFeature(
           remoting.ClientPlugin.Feature.INJECT_KEY_EVENT)) {
     var sendKeysElement = document.getElementById('send-keys-menu');
     sendKeysElement.hidden = true;
   } else if (this.mode_ != remoting.DesktopConnectedView.Mode.ME2ME &&
       this.mode_ != remoting.DesktopConnectedView.Mode.APP_REMOTING) {
     var sendCadElement = document.getElementById('send-ctrl-alt-del');
     sendCadElement.hidden = true;
   }
 
   // Apply customized key remappings if the plugin supports remapKeys.
   if (this.plugin_.hasFeature(remoting.ClientPlugin.Feature.REMAP_KEY)) {
     this.applyRemapKeys_(true);
   }
+
+  if (this.session_.hasCapability(
+          remoting.ClientSession.Capability.VIDEO_RECORDER)) {
+    this.videoFrameRecorder_ = new remoting.VideoFrameRecorder(this.plugin_);
+  }
 };
 
 /**
  * This is a callback that gets called when the window is resized.
  *
  * @return {void} Nothing.
  * @private.
  */
 remoting.DesktopConnectedView.prototype.onResize_ = function() {
   if (this.viewport_) {
     this.viewport_.onResize();
   }
 };
 
-/**
- * Callback that the plugin invokes to indicate when the connection is
- * ready.
- *
- * @param {boolean} ready True if the connection is ready.
- */
-remoting.DesktopConnectedView.prototype.onConnectionReady = function(ready) {
-  if (this.view_) {
-    this.view_.onConnectionReady(ready);
-  }
-};
-
-remoting.DesktopConnectedView.prototype.onConnected = function() {
+/** @private */
+remoting.DesktopConnectedView.prototype.initUI_ = function() {
   document.body.classList.add('connected');
 
   this.view_ = new remoting.ConnectedView(
       this.plugin_, this.container_,
       this.container_.querySelector('.mouse-cursor-overlay'));
 
   var scrollerElement = document.getElementById('scroller');
   this.viewport_ = new remoting.DesktopViewport(
       scrollerElement || document.body,
       this.plugin_.hostDesktop(),
       this.host_.options);
 
   if (remoting.windowFrame) {
     remoting.windowFrame.setDesktopConnectedView(this);
   }
   if (remoting.toolbar) {
     remoting.toolbar.setDesktopConnectedView(this);
   }
   if (remoting.optionsMenu) {
     remoting.optionsMenu.setDesktopConnectedView(this);
   }
 
   // Activate full-screen related UX.
   this.eventHooks_ = new base.Disposables(
     this.view_,
+    new base.EventHook(this.session_,
+                       remoting.ClientSession.Events.videoChannelStateChanged,
+                       this.view_.onConnectionReady.bind(this.view_)),
     new base.DomEventHook(window, 'resize', this.onResize_.bind(this), false),
-    new remoting.Fullscreen.EventHook(this.onFullScreenChanged_.bind(this))
-  );
+    new remoting.Fullscreen.EventHook(this.onFullScreenChanged_.bind(this)));
   this.onFullScreenChanged_(remoting.fullscreen.isActive());
 };
 
 /**
  * Set the shrink-to-fit and resize-to-client flags and save them if this is
  * a Me2Me connection.
  *
  * @param {boolean} shrinkToFit True if the remote desktop should be scaled
  *     down if it is larger than the client window; false if scroll-bars
  *     should be added in this case.
@@ skipping 112 matching lines @@
 /**
  * Sends a Print Screen keypress to the remoting client.
  *
  * @return {void} Nothing.
  */
 remoting.DesktopConnectedView.prototype.sendPrintScreen = function() {
   console.log('Sending Print Screen.');
   this.sendKeyCombination_([0x070046]);
 };
 
-remoting.DesktopConnectedView.prototype.initVideoFrameRecorder = function() {
-  this.videoFrameRecorder_ = new remoting.VideoFrameRecorder(this.plugin_);
-};
-
 /**
  * Returns true if the ClientSession can record video frames to a file.
  * @return {boolean}
  */
 remoting.DesktopConnectedView.prototype.canRecordVideo = function() {
   return !!this.videoFrameRecorder_;
 };
 
 /**
  * Returns true if the ClientSession is currently recording video frames.
@@ skipping 21 matching lines @@
  * @param {Object} message The parsed extension message data.
  * @return {boolean} True if the message was recognized, false otherwise.
  */
 remoting.DesktopConnectedView.prototype.handleExtensionMessage =
     function(type, message) {
   if (this.videoFrameRecorder_) {
     return this.videoFrameRecorder_.handleMessage(type, message);
   }
   return false;
 };
-
-/**
- * Handles dirty region debug messages.
- *
- * @param {{rects:Array<Array<number>>}} region Dirty region of the latest
- *     frame.
- */
-remoting.DesktopConnectedView.prototype.handleDebugRegion = function(region) {

[kelvinp, 2015/03/17 01:06:17]

Moved to ConnectedView.

-  while (this.debugRegionContainer_.firstChild) {
-    this.debugRegionContainer_.removeChild(
-        this.debugRegionContainer_.firstChild);
-  }
-  if (region.rects) {
-    var rects = region.rects;
-    for (var i = 0; i < rects.length; ++i) {
-      var rect = document.createElement('div');
-      rect.classList.add('debug-region-rect');
-      rect.style.left = rects[i][0] + 'px';
-      rect.style.top = rects[i][1] +'px';
-      rect.style.width = rects[i][2] +'px';
-      rect.style.height = rects[i][3] + 'px';
-      this.debugRegionContainer_.appendChild(rect);
-    }
-  }
-};