Skip to content

Commit

Permalink
Merge pull request #9807 from brave/maxk-cr93fu-enable-kEnablePasswor…
Browse files Browse the repository at this point in the history
…dsAccountStorage

[cr93 followup] Removed disablement of kEnablePasswordsAccountStorage.
  • Loading branch information
bsclifton authored and mkarolin committed Aug 25, 2021
1 parent 72f50d6 commit e059ea9
Show file tree
Hide file tree
Showing 4 changed files with 41 additions and 2 deletions.
1 change: 0 additions & 1 deletion app/brave_main_delegate.cc
Original file line number Diff line number Diff line change
Expand Up @@ -233,7 +233,6 @@ bool BraveMainDelegate::BasicStartupComplete(int* exit_code) {
net::features::kFirstPartySets.name,
network::features::kTrustTokens.name,
network_time::kNetworkTimeServiceQuerying.name,
password_manager::features::kEnablePasswordsAccountStorage.name,
reading_list::switches::kReadLater.name,
#if defined(OS_ANDROID)
features::kWebNfc.name,
Expand Down
1 change: 0 additions & 1 deletion app/brave_main_delegate_browsertest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,6 @@ IN_PROC_BROWSER_TEST_F(BraveMainDelegateBrowserTest, DisabledFeatures) {
&net::features::kFirstPartySets,
&network::features::kTrustTokens,
&network_time::kNetworkTimeServiceQuerying,
&password_manager::features::kEnablePasswordsAccountStorage,
&reading_list::switches::kReadLater,
};

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
/* Copyright (c) 2021 The Brave Authors. All rights reserved.
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this file,
* You can obtain one at http://mozilla.org/MPL/2.0/. */

#include "components/password_manager/core/browser/sync_credentials_filter.h"

#define ShouldSave ShouldSave_ChromiumImpl
#include "../../../../../../components/password_manager/core/browser/sync_credentials_filter.cc"
#undef ShouldSave

namespace password_manager {

bool SyncCredentialsFilter::ShouldSave(const PasswordForm& form) const {
bool should_save = ShouldSave_ChromiumImpl(form);
if (!should_save && sync_util::IsGaiaCredentialPage(form.signon_realm)) {
return true;
}
return should_save;
}

} // namespace password_manager
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
/* Copyright (c) 2021 The Brave Authors. All rights reserved.
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this file,
* You can obtain one at http://mozilla.org/MPL/2.0/. */

#ifndef BRAVE_CHROMIUM_SRC_COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_SYNC_CREDENTIALS_FILTER_H_
#define BRAVE_CHROMIUM_SRC_COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_SYNC_CREDENTIALS_FILTER_H_

#include "components/password_manager/core/browser/credentials_filter.h"

#define ShouldSave \
ShouldSave_ChromiumImpl(const PasswordForm& form) const; \
bool ShouldSave

#include "../../../../../../components/password_manager/core/browser/sync_credentials_filter.h"

#undef ShouldSave

#endif // BRAVE_CHROMIUM_SRC_COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_SYNC_CREDENTIALS_FILTER_H_

0 comments on commit e059ea9

Please sign in to comment.