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

Unified Diff: remoting/webapp/options_menu.js

Issue 339613003: Remove the blue tool-bar for apps v2. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Made unit-tests side-effect-free. Created 6 years, 5 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/menu_button.js ('k') | remoting/webapp/remoting.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/options_menu.js
diff --git a/remoting/webapp/options_menu.js b/remoting/webapp/options_menu.js
index fa1b9fd0153dfbf622916c2213062128cfb12d88..ca6d6156bc5f43436c5e95bb05151bacd2a3c50a 100644
--- a/remoting/webapp/options_menu.js
+++ b/remoting/webapp/options_menu.js
@@ -13,19 +13,22 @@
var remoting = remoting || {};
/**
- * @param {HTMLElement} sendCtrlAltDel
- * @param {HTMLElement} sendPrtScrn
- * @param {HTMLElement} resizeToClient
- * @param {HTMLElement} shrinkToFit
- * @param {HTMLElement?} fullscreen
+ * @param {Element} sendCtrlAltDel
+ * @param {Element} sendPrtScrn
+ * @param {Element} resizeToClient
+ * @param {Element} shrinkToFit
+ * @param {Element} newConnection
+ * @param {Element?} fullscreen
* @constructor
*/
remoting.OptionsMenu = function(sendCtrlAltDel, sendPrtScrn,
- resizeToClient, shrinkToFit, fullscreen) {
+ resizeToClient, shrinkToFit,
+ newConnection, fullscreen) {
this.sendCtrlAltDel_ = sendCtrlAltDel;
this.sendPrtScrn_ = sendPrtScrn;
this.resizeToClient_ = resizeToClient;
this.shrinkToFit_ = shrinkToFit;
+ this.newConnection_ = newConnection;
this.fullscreen_ = fullscreen;
/**
* @type {remoting.ClientSession}
@@ -41,6 +44,8 @@ remoting.OptionsMenu = function(sendCtrlAltDel, sendPrtScrn,
'click', this.onResizeToClient_.bind(this), false);
this.shrinkToFit_.addEventListener(
'click', this.onShrinkToFit_.bind(this), false);
+ this.newConnection_.addEventListener(
+ 'click', this.onNewConnection_.bind(this), false);
if (this.fullscreen_) {
this.fullscreen_.addEventListener(
'click', this.onFullscreen_.bind(this), false);
@@ -94,6 +99,14 @@ remoting.OptionsMenu.prototype.onShrinkToFit_ = function() {
}
};
+remoting.OptionsMenu.prototype.onNewConnection_ = function() {
+ chrome.app.window.create('main.html', {
+ 'width': 800,
+ 'height': 600,
+ 'frame': 'none'
+ });
+};
+
remoting.OptionsMenu.prototype.onFullscreen_ = function() {
remoting.fullscreen.toggle();
};
« no previous file with comments | « remoting/webapp/menu_button.js ('k') | remoting/webapp/remoting.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698