[AppRemoting] Implement ConnectionDroppedDialog.

remoting/webapp/app_remoting/js/app_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 app remoting session.
  */
 
 /** @suppress {duplicate} */
@@ skipping 56 matching lines @@
       this.plugin_.hostDesktop(),
       remoting.HostDesktop.Events.shapeChanged,
       remoting.windowShape.setDesktopRects.bind(remoting.windowShape));
 
   var desktopSizeHook = new base.EventHook(
       this.plugin_.hostDesktop(),
       remoting.HostDesktop.Events.sizeChanged,
       this.onDesktopSizeChanged_.bind(this));
 
   /** @private */
-  this.disposables_ = new base.Disposables(
-      baseView, windowShapeHook, desktopSizeHook, this.contextMenu_);
+  this.disposables_ =
+      new base.Disposables(baseView, windowShapeHook, desktopSizeHook,
+                           this.contextMenu_, menuAdapter);

[Jamie, 2015/05/20 02:20:36]

Is this change related to this CL?

[kelvinp, 2015/05/20 18:12:59]

Because, this change allow us to restart the AppRemotingActivity., each
component must be able to shutdown cleanly.  We add the menuAdaptor to the list
of disposables so that it can clean up its context menu, or else ChromeOS will
complain double registration of the same menu item. 

 
   /** @private */
   this.supportsGoogleDrive_ = this.plugin_.hasCapability(
       remoting.ClientSession.Capability.GOOGLE_DRIVE);
 
   this.resizeHostToClientArea_();
   this.plugin_.extensions().register(this);
 };
 
 /**
@@ skipping 155 matching lines @@
   }).catch(remoting.Error.handler(function(/** remoting.Error */ error) {
     console.log('Failed to refresh access token: ' + error.toString());
   }));
 };
 
 // The access token last received from getNewToken. Saved to ensure that we
 // get a fresh token each time.
 var previousToken_ = '';
 
 })();