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

Unified Diff: remoting/webapp/app_remoting/js/window_activation_menu.js

Issue 1816653002: Remove app_remoting. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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
Index: remoting/webapp/app_remoting/js/window_activation_menu.js
diff --git a/remoting/webapp/app_remoting/js/window_activation_menu.js b/remoting/webapp/app_remoting/js/window_activation_menu.js
deleted file mode 100644
index 72039b7fdf578c4911b6f981e75e14b8fa59ddcc..0000000000000000000000000000000000000000
--- a/remoting/webapp/app_remoting/js/window_activation_menu.js
+++ /dev/null
@@ -1,89 +0,0 @@
-// 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
- * Class to update the application's context menu to include host-side windows
- * and to notify the host when one of these menu items is selected.
- */
-
-'use strict';
-
-/** @suppress {duplicate} */
-var remoting = remoting || {};
-
-/**
- * @param {remoting.ContextMenuAdapter} adapter
- * @constructor
- */
-remoting.WindowActivationMenu = function(adapter) {
- /** @private {remoting.SubmenuManager} */
- this.submenuManager_ = new remoting.SubmenuManager(
- adapter,
- chrome.i18n.getMessage(/*i18n-content*/'WINDOWS_SUBMENU_TITLE'),
- false);
- /** @private {function(string, string)} */
- this.sendExtensionMessage_ = base.doNothing;
-
- adapter.addListener(this.onContextMenu_.bind(this));
-};
-
-/**
- * Add a window to the application's context menu, or update the title of an
- * existing window.
- *
- * @param {number} id The window id.
- * @param {string} title The window title.
- */
-remoting.WindowActivationMenu.prototype.add = function(id, title) {
- this.submenuManager_.add(this.makeMenuId_(id), title);
- // TODO(jamiewalch): Once crbug.com/426283 is fixed, call drawAttention()
- // here if the window does not have focus.
-};
-
-/**
- * Remove a window from the application's context menu.
- *
- * @param {number} id The window id.
- */
-remoting.WindowActivationMenu.prototype.remove = function(id) {
- this.submenuManager_.remove(this.makeMenuId_(id));
-};
-
-/**
- * Create a menu id from the given window id.
- *
- * @param {number} windowId
- * @return {string}
- * @private
- */
-remoting.WindowActivationMenu.prototype.makeMenuId_ = function(windowId) {
- return 'window-' + windowId;
-};
-
-/** @param {function(string, string)} callback */
-remoting.WindowActivationMenu.prototype.setExtensionMessageSender =
- function(callback) {
- this.sendExtensionMessage_ = callback;
-};
-
-/**
- * Handle a click on the application's context menu.
- *
- * @param {OnClickData=} info
- * @private
- */
-remoting.WindowActivationMenu.prototype.onContextMenu_ = function(info) {
- var menuId = info.menuItemId.toString();
- var components = menuId.split('-');
- if (components.length == 2 &&
- this.makeMenuId_(parseInt(components[1], 10)) == menuId) {
- this.sendExtensionMessage_(
- 'activateWindow',
- JSON.stringify({ id: parseInt(components[1], 0) }));
- if (chrome.app.window.current().isMinimized()) {
- chrome.app.window.current().restore();
- }
- }
-};
« no previous file with comments | « remoting/webapp/app_remoting/js/submenu_manager.js ('k') | remoting/webapp/app_remoting/manifest_common.json.jinja2 » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698