base::EnumSet::Empty/Size deleted
These functions now match the case in std functions. Chromium change: https://chromium.googlesource.com/chromium/src/+/bda6288d5f00d7d00ae4b4274ef8743b6cbeb9cd commit bda6288d5f00d7d00ae4b4274ef8743b6cbeb9cd Author: Lei Zhang <thestig@chromium.org> Date: Fri Mar 29 23:16:03 2024 +0000 Remove base::EnumSet::Empty() and Size() Replace with empty() and size(), respectively. The lowercase method names work better with templated code that expects classes to have size(), similar to STL containers. Fix a typo along the way.
This commit is contained in:
@@ -27,7 +27,7 @@ void RecordEnabledTypes(bool sync_everything_enabled,
|
||||
selected_types.HasAll(all_brave_supported_types)) {
|
||||
// Sync All
|
||||
sample = EnabledTypesAnswer::kAllTypes;
|
||||
} else if (selected_types.Empty() ||
|
||||
} else if (selected_types.empty() ||
|
||||
selected_types ==
|
||||
UserSelectableTypeSet({UserSelectableType::kBookmarks})) {
|
||||
// Bookmarks or no types
|
||||
|
||||
@@ -592,7 +592,7 @@ TEST_F(BraveSyncServiceImplTest, OnlyBookmarksAfterSetup) {
|
||||
brave_sync_service_impl()->GetUserSettings()->IsSyncEverythingEnabled());
|
||||
auto selected_types =
|
||||
brave_sync_service_impl()->GetUserSettings()->GetSelectedTypes();
|
||||
EXPECT_EQ(selected_types.Size(), 1u);
|
||||
EXPECT_EQ(selected_types.size(), 1u);
|
||||
EXPECT_TRUE(selected_types.Has(UserSelectableType::kBookmarks));
|
||||
|
||||
OSCryptMocker::TearDown();
|
||||
|
||||
Reference in New Issue
Block a user