[Webapp Refactor] Clean up remoting.Clipboard.

remoting/webapp/crd/js/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
  * Implements a basic UX control for a connected remoting session.
  */
 
 /** @suppress {duplicate} */
@@ skipping 24 matching lines @@
 
   /** @private {Element} */
   this.debugRegionContainer_ =
       viewportElement.querySelector('.debug-region-container');
 
   var pluginElement = plugin.element();
 
   /** private */
   this.disposables_ = new base.Disposables(
     this.cursor_,
-    new base.DomEventHook(pluginElement, 'focus',
-                          this.onPluginGotFocus_.bind(this), false),

[kelvinp, 2015/04/09 00:28:26]

moved to clipboard.js

     new base.DomEventHook(pluginElement, 'blur',
                           this.onPluginLostFocus_.bind(this), false),
     new base.DomEventHook(document, 'visibilitychange',
-                          this.onVisibilityChanged_.bind(this), false)
+                          this.onVisibilityChanged_.bind(this), false),
+    new remoting.Clipboard(plugin)
   );
 
   // TODO(wez): Only allow mouse lock if the app has the pointerLock permission.
   // Enable automatic mouse-lock.
   if (this.plugin_.hasFeature(remoting.ClientPlugin.Feature.ALLOW_MOUSE_LOCK)) {
     this.plugin_.allowMouseLock();
   }
 
   pluginElement.focus();
 };
@@ skipping 23 matching lines @@
  * Callback that the plugin invokes to indicate when the connection is
  * ready.
  *
  * @param {boolean} ready True if the connection is ready.
  */
 remoting.ConnectedView.prototype.onConnectionReady = function(ready) {
   this.viewportElement_.classList.toggle('session-client-inactive', !ready);
 };
 
 /**
- * Callback function called when the plugin element gets focus.
- * @private
- */
-remoting.ConnectedView.prototype.onPluginGotFocus_ = function() {
-  remoting.clipboard.initiateToHost();
-};
-
-/**
  * Callback function called when the plugin element loses focus.
  * @private
  */
 remoting.ConnectedView.prototype.onPluginLostFocus_ = function() {
   // Release all keys to prevent them becoming 'stuck down' on the host.
   this.plugin_.releaseAllKeys();
 
   // Focus should stay on the element, not (for example) the toolbar.
   // Due to crbug.com/246335, we can't restore the focus immediately,
   // otherwise the plugin gets confused about whether or not it has focus.
@@ skipping 86 matching lines @@
 /**
  * @param {Event} event
  * @private
  */
 remoting.ConnectedView.Cursor.prototype.onMouseMoved_ = function(event) {
   this.cursorElement_.style.top = event.offsetY + 'px';
   this.cursorElement_.style.left = event.offsetX + 'px';
 };
 
 })();