chromeos: Fix OS settings "People" row when Account Manager is disabled

Make the first row of the People section not clickable when the account
manager is disabled (e.g. via flags or Finch).

Bug: 950007
Test: added to browser_tests
Change-Id: Ib9720f55fb60b526afe3ae40566678f603423320
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760463Reviewed-by: Lutz Justen <ljusten@chromium.org>
Commit-Queue: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#688533}

chrome/browser/resources/settings/chromeos/os_people_page/os_people_page.html

@@ -107,7 +107,7 @@
               </div>
               <div class="middle two-line no-min-width"
                   on-click="onAccountManagerTap_"
-                  actionable>
+                  actionable$="[[isAccountManagerEnabled_]]">
                 <div class="flex text-elide settings-box-text">
                   <span id="profile-name">[[profileName_]]</span>
                   <div id="profile-label" class="secondary">
@@ -115,6 +115,7 @@
                   </div>
                 </div>
                 <cr-icon-button class="subpage-arrow"
+                    hidden="[[!isAccountManagerEnabled_]]"
                     id="account-manager-subpage-trigger"
                     aria-label="$i18n{accountManagerSubMenuLabel}"
                     aria-describedby="profile-name"></cr-icon-button>

chrome/browser/resources/settings/chromeos/os_people_page/os_people_page.js

@@ -337,7 +337,9 @@ Polymer({
    * @private
    */
   onAccountManagerTap_: function(e) {
-    settings.navigateTo(settings.routes.ACCOUNT_MANAGER);
+    if (this.isAccountManagerEnabled_) {
+      settings.navigateTo(settings.routes.ACCOUNT_MANAGER);
+    }
   },
 
   /**

chrome/test/data/webui/settings/chromeos/os_people_page_test.js

@@ -130,6 +130,9 @@ cr.define('settings_people_page', function() {
           'pushedName', peoplePage.$$('#profile-name').textContent.trim());
       const newBg = peoplePage.$$('#profile-icon').style.backgroundImage;
       assertTrue(newBg.includes(iconDataUrl));
+
+      // Sub-page trigger is hidden.
+      assertTrue(peoplePage.$$('#account-manager-subpage-trigger').hidden);
     });
 
     test('GAIA name and picture, account manager enabled', async () => {
@@ -156,6 +159,14 @@ cr.define('settings_people_page', function() {
       assertEquals(
           'primary@gmail.com, +2 more accounts',
           peoplePage.$$('#profile-label').textContent.trim());
+
+      // Sub-page trigger is shown.
+      const subpageTrigger = peoplePage.$$('#account-manager-subpage-trigger');
+      assertFalse(subpageTrigger.hidden);
+
+      // Sub-page trigger navigates to Google account manager.
+      subpageTrigger.click();
+      assertEquals(settings.getCurrentRoute(), settings.routes.ACCOUNT_MANAGER);
     });
   });
 });