MD Settings: Add Google Play Store (Arc++) section

chrome/browser/resources/settings/settings_main/settings_main.js

Index: chrome/browser/resources/settings/settings_main/settings_main.js
diff --git a/chrome/browser/resources/settings/settings_main/settings_main.js b/chrome/browser/resources/settings/settings_main/settings_main.js
index 5d9dd81c119ed6873abf7b03e5335da9e74d14f7..6fcdf96c907d468cb6821216648d669499b29868 100644
--- a/chrome/browser/resources/settings/settings_main/settings_main.js
+++ b/chrome/browser/resources/settings/settings_main/settings_main.js
@@ -93,6 +93,8 @@ Polymer({
     this.listen(this, 'freeze-scroll', 'onFreezeScroll_');
     var currentRoute = settings.getCurrentRoute();
     this.hasExpandedSection_ = currentRoute && currentRoute.isSubpage();
+    if (!loadTimeData.getBoolean('androidAppsAllowed'))
+        this.pageVisibility.androidApps = false;

[michaelpg, 2016/12/01 20:15:34]

does this work? i'm worried about a couple things:

* it's not using this.set, so settings-ui and settings-menu are not notified of
the change (e.g., it won't hide it from the menu)
* it's happening dynamically, so i'm not convinced the page doesn't get stamped
first before hiding
* should we hide this in guest mode via settings_ui.js

[stevenjb, 2016/12/02 00:39:19]

Ugh. The way we handle pageVisibility is confusing. We use it one way for tests,
a different way for gues tmode.

Rather than make a major change to that code, I will just add a separate
property in settings_ui instead. (This was the wrong place for this regardless,
I didn't realize settings_ui was at the top of the tree).

[michaelpg, 2016/12/02 03:47:43]

Yeah... I don't know if there's a non-sucky way of doing this without fixing
pageVisibility.

   },
 
   /** @override */