From 177169d20749448fd51e0fa73fb526c973947320 Mon Sep 17 00:00:00 2001
From: Thomas Tangl <tangltom@chromium.org>
Date: Thu, 20 Dec 2018 14:18:43 +0000
Subject: [PATCH] [unified-consent] Navigate away from sync controls page when
 unavailable

- Navigate to main sync page when sync controls page should *not* be
  available. I.e. when sync is paused, the user is signed out, sync
  is disabled or there is a sync error other than passphrase error.

- Don't show sync controls page when unified consent is disabled.

Bug: 915744
Change-Id: I762a91b170ad379c2ffb8fe9eab4f42a6414fcd0
Reviewed-on: https://chromium-review.googlesource.com/c/1380676
Reviewed-by: Scott Chen <scottchen@chromium.org>
Commit-Queue: Thomas Tangl <tangltom@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#617407}(cherry picked from commit 7db78b069b5fc71f6023ba401690b60292390747)
Reviewed-on: https://chromium-review.googlesource.com/c/1386826
Reviewed-by: Thomas Tangl <tangltom@chromium.org>
Cr-Commit-Position: refs/branch-heads/3626@{#481}
Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437}
---
 .../settings/people_page/people_page.html     | 16 +++++----
 .../settings/people_page/sync_controls.js     | 22 +++++++++++-
 .../people_page_sync_controls_test.js         | 34 +++++++++++++++++--
 3 files changed, 62 insertions(+), 10 deletions(-)

diff --git a/chrome/browser/resources/settings/people_page/people_page.html b/chrome/browser/resources/settings/people_page/people_page.html
index ccbbfb55030ce..66d24d1f29d44 100644
--- a/chrome/browser/resources/settings/people_page/people_page.html
+++ b/chrome/browser/resources/settings/people_page/people_page.html
@@ -340,13 +340,15 @@
         </settings-subpage>
       </template>
 
-      <template is="dom-if" route-path="/syncSetup/advanced">
-        <settings-subpage page-title="$i18n{syncAdvancedPageTitle}"
-            learn-more-url="$i18n{syncAndGoogleServicesLearnMoreURL}">
-          <settings-sync-controls sync-status="[[syncStatus]]"
-              on-sync-setup-cancel="cancelSyncSetup_">
-          </settings-sync-controls>
-        </settings-subpage>
+      <template is="dom-if" if="[[unifiedConsentEnabled_]]">
+        <template is="dom-if" route-path="/syncSetup/advanced">
+          <settings-subpage page-title="$i18n{syncAdvancedPageTitle}"
+              learn-more-url="$i18n{syncAndGoogleServicesLearnMoreURL}">
+            <settings-sync-controls sync-status="[[syncStatus]]"
+                on-sync-setup-cancel="cancelSyncSetup_">
+            </settings-sync-controls>
+          </settings-subpage>
+        </template>
       </template>
 
 <if expr="chromeos">
diff --git a/chrome/browser/resources/settings/people_page/sync_controls.js b/chrome/browser/resources/settings/people_page/sync_controls.js
index 859f88745c138..d910e9fbf58f0 100644
--- a/chrome/browser/resources/settings/people_page/sync_controls.js
+++ b/chrome/browser/resources/settings/people_page/sync_controls.js
@@ -42,7 +42,10 @@ Polymer({
      * The current sync status, supplied by the parent.
      * @type {settings.SyncStatus}
      */
-    syncStatus: Object,
+    syncStatus: {
+      type: Object,
+      observer: 'syncStatusChanged_',
+    },
   },
 
   /** @private {?settings.SyncBrowserProxy} */
@@ -168,5 +171,22 @@ Polymer({
   onCancelSyncClick_: function() {
     this.fire('sync-setup-cancel');
   },
+
+  /** @private */
+  syncStatusChanged_: function(syncStatus) {
+    // When the sync controls are embedded, the parent has to take care of
+    // showing/hiding them.
+    if (settings.getCurrentRoute() != settings.routes.SYNC_ADVANCED ||
+        !syncStatus)
+      return;
+
+    // Navigate to main sync page when the sync controls page should *not* be
+    // available.
+    if (!syncStatus.signedIn || !!syncStatus.disabled ||
+        (!!syncStatus.hasError &&
+         syncStatus.statusAction !== settings.StatusAction.ENTER_PASSPHRASE)) {
+      settings.navigateTo(settings.routes.SYNC);
+    }
+  },
 });
 })();
diff --git a/chrome/test/data/webui/settings/people_page_sync_controls_test.js b/chrome/test/data/webui/settings/people_page_sync_controls_test.js
index 7eac1b4d00e0c..e5b902ebfb9f1 100644
--- a/chrome/test/data/webui/settings/people_page_sync_controls_test.js
+++ b/chrome/test/data/webui/settings/people_page_sync_controls_test.js
@@ -131,7 +131,11 @@ cr.define('settings_people_page_sync_controls', function() {
       settings.navigateTo(settings.routes.SYNC_ADVANCED);
       document.body.appendChild(syncControls);
 
+      syncControls
+          .syncStatus = {disabled: false, hasError: false, signedIn: true};
       Polymer.dom.flush();
+
+      assertEquals(settings.routes.SYNC_ADVANCED, settings.getCurrentRoute());
     });
 
     teardown(function() {
@@ -140,14 +144,40 @@ cr.define('settings_people_page_sync_controls', function() {
 
     if (!cr.isChromeOS) {
       test('ToastShownForSyncSetup', function() {
-        syncControls.syncStatus = {setupInProgress: false};
+        syncControls.syncStatus = {setupInProgress: false, signedIn: true};
         assertFalse(syncControls.$.toast.open);
 
-        syncControls.syncStatus = {setupInProgress: true};
+        syncControls.syncStatus = {setupInProgress: true, signedIn: true};
         assertTrue(syncControls.$.toast.open);
 
         syncControls.$.toast.querySelector('paper-button').click();
       });
     }
+
+    test('SignedOut', function() {
+      syncControls
+          .syncStatus = {disabled: false, hasError: false, signedIn: false};
+      assertEquals(settings.routes.SYNC, settings.getCurrentRoute());
+    });
+
+    test('PassphraseError', function() {
+      syncControls.syncStatus = {
+        disabled: false,
+        hasError: true,
+        signedIn: true,
+        statusAction: settings.StatusAction.ENTER_PASSPHRASE
+      };
+      assertEquals(settings.routes.SYNC_ADVANCED, settings.getCurrentRoute());
+    });
+
+    test('SyncPaused', function() {
+      syncControls.syncStatus = {
+        disabled: false,
+        hasError: true,
+        signedIn: true,
+        statusAction: settings.StatusAction.REAUTHENTICATE
+      };
+      assertEquals(settings.routes.SYNC, settings.getCurrentRoute());
+    });
   });
 });