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

Unified Diff: remoting/webapp/crd/js/background.js

Issue 868203002: Handle authentication failures in the v2 app by restarting the app (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 11 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/crd/js/app_launcher.js ('k') | remoting/webapp/crd/js/crd_event_handlers.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/crd/js/background.js
diff --git a/remoting/webapp/crd/js/background.js b/remoting/webapp/crd/js/background.js
index f3d5bea5c98d5ece80db81bb4df89631552f72fc..fd2e41de623798d6bcdf503e9266d0f72ceb3bc4 100644
--- a/remoting/webapp/crd/js/background.js
+++ b/remoting/webapp/crd/js/background.js
@@ -7,57 +7,7 @@ var remoting = remoting || {};
(function(){
-/**
- * A class that handles application activation.
- *
- * @param {remoting.AppLauncher} appLauncher
- * @constructor
- */
-function ActivationHandler(appLauncher) {
- /**
- * @type {remoting.AppLauncher}
- * @private
- */
- this.appLauncher_ = appLauncher;
-
- chrome.contextMenus.create({
- id: ActivationHandler.NEW_WINDOW_MENU_ID_,
- contexts: ['launcher'],
- title: chrome.i18n.getMessage(/*i18n-content*/'NEW_WINDOW')
- });
-
- chrome.contextMenus.onClicked.addListener(this.onContextMenu_.bind(this));
- chrome.app.runtime.onLaunched.addListener(this.onLaunched_.bind(this));
-}
-
-/** @type {string} */
-ActivationHandler.NEW_WINDOW_MENU_ID_ = 'new-window';
-
-/**
- * @param {OnClickData} info
- * @private
- */
-ActivationHandler.prototype.onContextMenu_ = function(info) {
- if (info.menuItemId == ActivationHandler.NEW_WINDOW_MENU_ID_) {
- this.appLauncher_.launch();
- }
-};
-
-/**
- * Called when the App is activated (e.g. from the Chrome App Launcher). It
- * creates a new window if there are no existing ones. Otherwise, it will put
- * focus on the last window created.
- *
- * @private
- */
-ActivationHandler.prototype.onLaunched_ = function() {
- var windows = chrome.app.window.getAll();
- if (windows.length >= 1) {
- windows[windows.length - 1].focus();
- } else {
- this.appLauncher_.launch();
- }
-};
+'use strict';
/**
* The background service is responsible for listening to incoming connection
@@ -86,7 +36,8 @@ function initializeBackgroundService(appLauncher) {
function main() {
if (base.isAppsV2()) {
- new ActivationHandler(new remoting.V2AppLauncher());
+ new remoting.ActivationHandler(base.Ipc.getInstance(),
+ new remoting.V2AppLauncher());
}
}
« no previous file with comments | « remoting/webapp/crd/js/app_launcher.js ('k') | remoting/webapp/crd/js/crd_event_handlers.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698