arc: Handle ARC events in MD Settings

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

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..0f8c4aca707da19b828dbb3c52d0ed370bf63689 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
@@ -54,7 +54,14 @@ Polymer({
    * @private
    */
   androidAppsInfoUpdate_: function(info) {
+    if (!this.parentElement)

[khmel, 2017/05/10 21:48:13]

in test this causes multiple listener invocation and as result multiple
navigateToPreviousRoute (which is async)

[stevenjb, 2017/05/10 22:35:44]

This is weird, we should avoid it. The test to settings.getCurrentRoute() should
be sufficient?

If not, try adding an observer to androidAppsInfo in the subpage and moving the
test there?

In practice, we don't necessarily need this. Any UI action that would cause this
change could close the details page, and if playStoreEnabled changes to false
while the details page is shown, it's not the end of the world if it remains
open.

[khmel, 2017/05/11 00:10:30]

settings.navigateToPreviousRoute() is async (windows.back)
so settings.getCurrentRoute() == settings.Route.ANDROID_APPS_DETAILS) will be
true several times.

it's not the end of the world but the end of the ideal world :)
Frankly speaking that looks really weird. I can close ARC OptIn window. This
disables ARC but settings still show "Remove Android Apps"
We have to handle this case.

[stevenjb, 2017/05/11 16:58:14]

When would that ever happen in practice?

Again, the best solution is probably to put the logic in the sub page instead.

[khmel, 2017/05/11 17:32:22]

I described when this happens above. For example open details page. Disable ARC
not from settings, for example closing ARC OptIn window when it shows start
page, LSO page or error page. ARC is disabled not but details settings are
shown.

Moved it to subpage.

+      return;
+
     this.androidAppsInfo_ = info;
+    if (!this.androidAppsInfo_.playStoreEnabled &&
+        settings.getCurrentRoute() == settings.Route.ANDROID_APPS_DETAILS) {
+      settings.navigateToPreviousRoute();
+    }
   },
 
   /**
@@ -68,7 +75,7 @@ Polymer({
 
   /** @private */
   onSubpageTap_: function() {
-    if (this.androidAppsInfo_.appReady)
+    if (this.androidAppsInfo_.playStoreEnabled)
       settings.navigateTo(settings.Route.ANDROID_APPS_DETAILS);
   },
 });