Remove the blue tool-bar for apps v2.

remoting/webapp/window_frame.js

 // Copyright 2014 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
  * Apps v2 custom title bar implementation
  */
 
 'use strict';
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 /**
  * @param {HTMLElement} titleBar The root node of the title-bar DOM hierarchy.
  * @constructor
  */
 remoting.WindowFrame = function(titleBar) {
   /**
-   * @type {boolean}
+   * @type {remoting.ClientSession}
    * @private
    */
-  this.isConnected_ = false;
+  this.clientSession_ = null;
 
   /**
    * @type {HTMLElement}
    * @private
    */
   this.titleBar_ = titleBar;
 
   /**
    * @type {HTMLElement}
    * @private
    */
   this.hoverTarget_ = /** @type {HTMLElement} */
       (titleBar.querySelector('.window-controls-hover-target'));
   base.debug.assert(this.hoverTarget_ != null);
 
   /**
+   * @type {remoting.OptionsMenu}
+   * @private
+   */
+  this.optionsMenu_ = new remoting.OptionsMenu(
+      titleBar.querySelector('.menu-send-ctrl-alt-del'),
+      titleBar.querySelector('.menu-send-print-screen'),
+      titleBar.querySelector('.menu-resize-to-client'),
+      titleBar.querySelector('.menu-shrink-to-fit'),
+      titleBar.querySelector('.menu-new-connection'),
+      null);
+
+  /**
    * @type {HTMLElement}
    * @private
    */
+  this.title_ = /** @type {HTMLElement} */
+      (titleBar.querySelector('.window-title'));
+  base.debug.assert(this.title_ != null);
+
+  /**
+   * @type {HTMLElement}
+   * @private
+   */
   this.maximizeRestoreControl_ = /** @type {HTMLElement} */
       (titleBar.querySelector('.window-maximize-restore'));
   base.debug.assert(this.maximizeRestoreControl_ != null);
 
+  var optionsButton = titleBar.querySelector('.window-options');
+  base.debug.assert(optionsButton != null);
+  this.optionMenuButton_ = new remoting.MenuButton(
+      optionsButton,
+      this.onShowOptionsMenu_.bind(this),
+      this.onHideOptionsMenu_.bind(this));
+
   /**
    * @type {Array.<{cls:string, fn: function()}>}
    */
   var handlers = [
     { cls: 'window-disconnect', fn: this.disconnectSession_.bind(this) },
     { cls: 'window-maximize-restore',
       fn: this.maximizeOrRestoreWindow_.bind(this) },
     { cls: 'window-minimize', fn: this.minimizeWindow_.bind(this) },
     { cls: 'window-close', fn: window.close.bind(window) },
     { cls: 'window-controls-stub', fn: this.toggleWindowControls_.bind(this) }
   ];
   for (var i = 0; i < handlers.length; ++i) {
     var element = titleBar.querySelector('.' + handlers[i].cls);
     base.debug.assert(element != null);
     element.addEventListener('click', handlers[i].fn, false);
   }
 
   // Ensure that tool-tips are always correct.
   this.updateMaximizeOrRestoreIconTitle_();
   chrome.app.window.current().onMaximized.addListener(
       this.updateMaximizeOrRestoreIconTitle_.bind(this));
   chrome.app.window.current().onRestored.addListener(
       this.updateMaximizeOrRestoreIconTitle_.bind(this));
   chrome.app.window.current().onFullscreened.addListener(
       this.updateMaximizeOrRestoreIconTitle_.bind(this));
 };
 
 /**
- * @param {boolean} isConnected True if there is a connection active.
+ * @param {remoting.ClientSession} clientSession The client session, or null if
+ *     there is no connection.
  */
-remoting.WindowFrame.prototype.setConnected = function(isConnected) {
-  this.isConnected_ = isConnected;
-  if (this.isConnected_) {
+remoting.WindowFrame.prototype.setClientSession = function(clientSession) {
+  this.optionsMenu_.setClientSession(clientSession);
+  this.clientSession_ = clientSession;

[kelvinp, 2014/07/23 18:49:08]

It seems like clientSession_ is only used as a boolean.

[Jamie, 2014/07/23 20:00:38]

We need a ClientSession in order to set the window title. You're correct that we
don't actually need to store the client session, but as discussed, it doesn't
really make a lot of difference either way so I'm going to leave it as-is.

+  var windowTitle = document.head.querySelector('title');
+  if (this.clientSession_) {
     document.body.classList.add('connected');
+    this.title_.innerText = clientSession.getHostDisplayName();
+    windowTitle.innerText = clientSession.getHostDisplayName() + ' - ' +
+        chrome.i18n.getMessage(/*i18n-content*/'PRODUCT_NAME');
   } else {
     document.body.classList.remove('connected');
+    this.title_.innerHTML = ' ';
+    windowTitle.innerText =
+        chrome.i18n.getMessage(/*i18n-content*/'PRODUCT_NAME');
   }
   this.updateMaximizeOrRestoreIconTitle_();
 };
 
 /**
  * @return {{width: number, height: number}} The size of the window, ignoring
  *     the title-bar and window borders, if visible.
  */
 remoting.WindowFrame.prototype.getClientArea = function() {
   if (chrome.app.window.current().isFullscreen()) {
@@ skipping 28 matching lines @@
 remoting.WindowFrame.prototype.maximizeOrRestoreWindow_ = function() {
   /** @type {boolean} */
   var restore =
       chrome.app.window.current().isFullscreen() ||
       chrome.app.window.current().isMaximized();
   if (restore) {
     // Restore twice: once to exit full-screen and once to exit maximized.
     // If the app is not full-screen, or went full-screen without first
     // being maximized, then the second restore has no effect.
     chrome.app.window.current().restore();
-  } else if (this.isConnected_) {
+    chrome.app.window.current().restore();
+  } else if (this.clientSession_) {
     chrome.app.window.current().fullscreen();
   } else {
     chrome.app.window.current().maximize();
   }
 };
 
 /**
  * @private
  */
 remoting.WindowFrame.prototype.minimizeWindow_ = function() {
@@ skipping 13 matching lines @@
  *
  * @private
  */
 remoting.WindowFrame.prototype.updateMaximizeOrRestoreIconTitle_ = function() {
   /** @type {string} */
   var tag = '';
   if (chrome.app.window.current().isFullscreen()) {
     tag = /*i18n-content*/'EXIT_FULL_SCREEN';
   } else if (chrome.app.window.current().isMaximized()) {
     tag = /*i18n-content*/'RESTORE_WINDOW';
-  } else if (this.isConnected_) {
+  } else if (this.clientSession_) {
     tag = /*i18n-content*/'FULL_SCREEN';
   } else {
     tag = /*i18n-content*/'MAXIMIZE_WINDOW';
   }
   this.maximizeRestoreControl_.title = l10n.getTranslationOrError(tag);
 };
 
+/**
+ * Callback invoked when the options menu is shown.
+ * @private
+ */
+remoting.WindowFrame.prototype.onShowOptionsMenu_ = function() {
+  this.optionsMenu_.onShow();
+  this.hoverTarget_.classList.add('menu-opened');
+};
+
+/**
+ * Callback invoked when the options menu is shown.
+ * @private
+ */
+remoting.WindowFrame.prototype.onHideOptionsMenu_ = function() {
+  this.hoverTarget_.classList.remove('menu-opened');
+};
+
+
 /** @type {remoting.WindowFrame} */
-remoting.windowFrame = null;
+remoting.windowFrame = null;