arc: Handle ARC events in MD Settings

chrome/browser/resources/settings/android_apps_page/android_apps_page.js

 // Copyright 2016 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
  * 'android-apps-page' is the settings page for enabling android apps.
  */
 
 Polymer({
@@ skipping 27 matching lines @@
   /** @private {?settings.AndroidAppsBrowserProxy} */
   browserProxy_: null,
 
   /** @override */
   created: function() {
     this.browserProxy_ = settings.AndroidAppsBrowserProxyImpl.getInstance();
   },
 
   /** @override */
   ready: function() {
-    cr.addWebUIListener(
+    this.listener_ = cr.addWebUIListener(
         'android-apps-info-update', this.androidAppsInfoUpdate_.bind(this));
     this.browserProxy_.requestAndroidAppsInfo();
   },
+
   /**
    * @param {AndroidAppsInfo} info
    * @private
    */
   androidAppsInfoUpdate_: function(info) {
     this.androidAppsInfo_ = info;
   },
 
   /**
    * @param {Event} event
    * @private
    */
   onEnableTap_: function(event) {
     this.setPrefValue('arc.enabled', true);
     event.stopPropagation();
   },
 
   /** @private */
   onSubpageTap_: function() {
-    if (this.androidAppsInfo_.appReady)
+    if (this.androidAppsInfo_.playStoreEnabled)
       settings.navigateTo(settings.Route.ANDROID_APPS_DETAILS);
   },
+
+  /**
+   * Discards current element from handling android-apps-info-update events.
+   */
+  discardForTest: function() {
+    if (this.listener_) {
+      cr.removeWebUIListener(this.listener_);
+      this.listener_ = null;
+    }
+  },

[stevenjb, 2017/05/11 19:40:43]

There is a better way to do this, see comments in test.
(BTW, if we did do this we would also need to declare listener_).

[khmel, 2017/05/11 20:26:42]

Done as chatted.

 });