Accept empty username as sync username in IsSyncAccountCredential

components/password_manager/sync/browser/password_sync_util.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/password_manager/sync/browser/password_sync_util.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/common/password_form.h"
 #include "google_apis/gaia/gaia_auth_util.h"
 #include "google_apis/gaia/gaia_urls.h"
@@ skipping 21 matching lines @@
 }
 
 bool IsSyncAccountCredential(const autofill::PasswordForm& form,
                              const sync_driver::SyncService* sync_service,
                              const SigninManagerBase* signin_manager) {
   const Origin gaia_origin(GaiaUrls::GetInstance()->gaia_url().GetOrigin());
   if (!Origin(GURL(form.signon_realm)).IsSameOriginWith(gaia_origin)) {
     return false;
   }
 
+  // The empty username can mean that Chrome did not detect it correctly. For
+  // reasons described in http://crbug.com/636292#c1, the username is suspected
+  // to be the sync username unless proven otherwise.
+  if (form.username_value.empty())
+    return true;
+
   return gaia::AreEmailsSame(
       base::UTF16ToUTF8(form.username_value),
       GetSyncUsernameIfSyncingPasswords(sync_service, signin_manager));
 }
 
 }  // namespace sync_util
 }  // namespace password_manager