Respect opt_scopes in getNewToken.

remoting/webapp/app_remoting/js/app_connected_view.js

Index: remoting/webapp/app_remoting/js/app_connected_view.js
diff --git a/remoting/webapp/app_remoting/js/app_connected_view.js b/remoting/webapp/app_remoting/js/app_connected_view.js
index 73c308840c75abfff56082c91db00444f2c314d0..34a195137371225933da5e1d53d9ee12662535ca 100644
--- a/remoting/webapp/app_remoting/js/app_connected_view.js
+++ b/remoting/webapp/app_remoting/js/app_connected_view.js
@@ -238,11 +238,17 @@ remoting.AppConnectedView.prototype.sendGoogleDriveAccessToken_ =
     'https://www.googleapis.com/auth/drive'
   ];
   remoting.identity.getNewToken(googleDriveScopes).then(
-    function(/** string */ token){
+    function(/** string */ token) {
+      base.debug.assert(token !== previousToken_);
+      previousToken_ = token;
       sendExtensionMessage('accessToken', token);
   }).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_ = '';

[kelvinp, 2015/05/11 17:17:33]

This can be moved inside the scope of sendGoogleDriveAccessToken_

[Jamie, 2015/05/11 17:23:08]

I think it's clearer as a global. Javascript scoping and lifetime rules for vars
are a bit unusual to someone not well versed in the language, and since we're in
a IIFE, there's little benefit to scoping it more tightly than this.

+
 })();