Chromium Code Reviews| Index: chrome/browser/resources/settings/android_apps_page/android_apps_page.js |
| diff --git a/chrome/browser/resources/settings/android_apps_page/android_apps_page.js b/chrome/browser/resources/settings/android_apps_page/android_apps_page.js |
| index 861731780652ab51607d5907c609012ddb6e61da..8369aad32104f8abb0430507298ed362d30fbd5c 100644 |
| --- a/chrome/browser/resources/settings/android_apps_page/android_apps_page.js |
| +++ b/chrome/browser/resources/settings/android_apps_page/android_apps_page.js |
| @@ -33,6 +33,9 @@ Polymer({ |
| return map; |
| }, |
| }, |
| + |
| + /** @private {!WebUIListener} */ |
|
stevenjb
2017/05/11 20:33:51
This can just be a member variable like browserPro
khmel
2017/05/11 20:38:11
Done.
|
| + listener_: Object, |
| }, |
| /** @private {?settings.AndroidAppsBrowserProxy} */ |
| @@ -44,11 +47,17 @@ Polymer({ |
| }, |
| /** @override */ |
| - ready: function() { |
| - cr.addWebUIListener( |
| + attached: function() { |
| + this.listener_ = cr.addWebUIListener( |
| 'android-apps-info-update', this.androidAppsInfoUpdate_.bind(this)); |
| this.browserProxy_.requestAndroidAppsInfo(); |
| }, |
| + |
| + /** @override */ |
| + detached: function() { |
| + cr.removeWebUIListener(this.listener_); |
| + }, |
| + |
| /** |
| * @param {AndroidAppsInfo} info |
| * @private |
| @@ -68,7 +77,7 @@ Polymer({ |
| /** @private */ |
| onSubpageTap_: function() { |
| - if (this.androidAppsInfo_.appReady) |
| + if (this.androidAppsInfo_.playStoreEnabled) |
| settings.navigateTo(settings.Route.ANDROID_APPS_DETAILS); |
| }, |
| }); |