[Passwords] Suppress leak check for sync users with too many passwords

This CL doesn't allow sync users to trigger the local leak check if they
have too many passwords to complete the check.
They can circumvent this restriction by turning off sync temporarily.

Screenshot of the new state:
https://drive.google.com/a/google.com/file/d/1vwa-7a_pbp-hNbEeFX6K6_ePus4kjzZS/view?usp=sharing

Bug: 1047726, 1061109
Change-Id: Ic3de634d767ca11ebd85b6c217ca5d8985da606c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100789
Commit-Queue: Friedrich [CET] <fhorschig@chromium.org>
Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#749790}

chrome/browser/resources/settings/autofill_page/password_check.html

@@ -93,7 +93,7 @@
       </div>
       <cr-button id="controlPasswordCheckButton"
           on-click="onPasswordCheckButtonClick_"
-          hidden$="[[isButtonHidden_(status_)]]">
+          hidden$="[[isButtonHidden_(status_, suppressCheckupLink_)]]">
         [[getButtonText_(status_)]]
       </cr-button>
     </div>

chrome/browser/resources/settings/autofill_page/password_check.js

@@ -332,8 +332,9 @@ Polymer({
       case CheckState.SIGNED_OUT:
       case CheckState.NO_PASSWORDS:
       case CheckState.OTHER_ERROR:
-      case CheckState.TOO_MANY_PASSWORDS:
         return false;
+      case CheckState.TOO_MANY_PASSWORDS:
+        return !this.suppressesCheckupLink_();
       case CheckState.TOO_MANY_PASSWORDS_AND_QUOTA_LIMIT:
       case CheckState.QUOTA_LIMIT:
         return true;

chrome/test/data/webui/settings/password_check_test.js

@@ -153,9 +153,9 @@ cr.define('settings_passwords_check', function() {
       });
     });
 
-    // Test verifies that too many passwords don't prevent the user from
-    // starting the check. Additionally, display a link to account checkup.
-    test('testCheckupLinkForTooManyPasswordsWhenSyncing', function() {
+    // Test verifies that sync users see only the link to account checkup and no
+    // button to start the local leak check.
+    test('testOnlyCheckupLinkForTooManyPasswordsWhenSyncing', function() {
       passwordManager.data.checkStatus =
           autofill_test_util.makePasswordCheckStatus(
               /*state=*/ PasswordCheckState.TOO_MANY_PASSWORDS);
@@ -167,10 +167,7 @@ cr.define('settings_passwords_check', function() {
 
       return passwordManager.whenCalled('getPasswordCheckStatus').then(() => {
         expectTrue(isElementVisible(section.$.linkToGoogleAccount));
-        assertTrue(isElementVisible(section.$.controlPasswordCheckButton));
-        expectEquals(
-            section.i18n('checkPasswordsAgain'),
-            section.$.controlPasswordCheckButton.innerText);
+        expectFalse(isElementVisible(section.$.controlPasswordCheckButton));
       });
     });