Chromium Code Reviews| Index: remoting/webapp/crd/js/apps_v2_migration.js |
| diff --git a/remoting/webapp/crd/js/apps_v2_migration.js b/remoting/webapp/crd/js/apps_v2_migration.js |
| index 3f49883f2b26f624b1874ab0ba5733187265e66c..aab9994badb23d7b6a018c6d522ef21b86d5c7ad 100644 |
| --- a/remoting/webapp/crd/js/apps_v2_migration.js |
| +++ b/remoting/webapp/crd/js/apps_v2_migration.js |
| @@ -22,9 +22,9 @@ var remoting = remoting || {}; |
| var MIGRATION_KEY_ = 'remoting-v2-migration'; |
| /** |
| - * @constructor |
| * @param {string} email |
| * @param {string} fullName |
| + * @constructor |
| */ |
| remoting.MigrationSettings = function(email, fullName) { |
| this.email = email; |
| @@ -88,16 +88,14 @@ remoting.AppsV2Migration.saveUserInfo = function() { |
| if (base.isAppsV2()) { |
| chrome.storage.local.remove(MIGRATION_KEY_); |
| } else { |
| - /** |
| - * @param {string} email |
| - * @param {string} fullName |
| - */ |
| - remoting.identity.getUserInfo().then(function(userInfo) { |
| - var preference = {}; |
| - preference[MIGRATION_KEY_] = |
| - new remoting.MigrationSettings(userInfo.email, userInfo.name); |
| - chrome.storage.local.set(preference); |
| - }).catch(base.doNothing); |
| + remoting.identity.getUserInfo().then( |
| + /** @param {{email:string, name:string}} userInfo */ |
|
kelvinp
2015/02/28 03:03:52
Nit: Don't think we need the JSDoc at all, as the
garykac
2015/02/28 03:28:23
This was added at the request of jscompile.
|
| + function(userInfo) { |
| + var preference = {}; |
| + preference[MIGRATION_KEY_] = |
| + new remoting.MigrationSettings(userInfo.email, userInfo.name); |
| + chrome.storage.local.set(preference); |
| + }).catch(base.doNothing); |
| } |
| }; |