Simplify some base::EnumSet::Size() calls
Replace `Size() == 0` with `empty()`. In idle_timeout_policy_handler.cc,
also remove a follow-up Size() call that always evaluates to true.
Change-Id: Ic5f91fa2555a15f5b7d50008cda2f19002cb2c9a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5405715
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Owners-Override: Peter Kasting <pkasting@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1280408}
diff --git a/chrome/browser/browsing_data/browsing_data_lifetime_policy_handler.cc b/chrome/browser/browsing_data/browsing_data_lifetime_policy_handler.cc
index b012631..3ab2fe0 100644
--- a/chrome/browser/browsing_data/browsing_data_lifetime_policy_handler.cc
+++ b/chrome/browser/browsing_data/browsing_data_lifetime_policy_handler.cc
@@ -94,7 +94,7 @@
void BrowsingDataLifetimePolicyHandler::PrepareForDisplaying(
policy::PolicyMap* policies) const {
policy::PolicyMap::Entry* entry = policies->GetMutable(policy_name());
- if (!entry || forced_disabled_sync_types_.Size() == 0) {
+ if (!entry || forced_disabled_sync_types_.empty()) {
return;
}
diff --git a/components/enterprise/idle/idle_timeout_policy_handler.cc b/components/enterprise/idle/idle_timeout_policy_handler.cc
index 177b224..9a8831e 100644
--- a/components/enterprise/idle/idle_timeout_policy_handler.cc
+++ b/components/enterprise/idle/idle_timeout_policy_handler.cc
@@ -225,17 +225,16 @@
void IdleTimeoutActionsPolicyHandler::PrepareForDisplaying(
policy::PolicyMap* policies) const {
policy::PolicyMap::Entry* entry = policies->GetMutable(policy_name());
- if (!entry || forced_disabled_sync_types_.Size() == 0) {
+ if (!entry || forced_disabled_sync_types_.empty()) {
return;
}
// `PolicyConversionsClient::GetPolicyValue()` doesn't support
// MessageType::kInfo in the PolicyErrorMap, so add the message to the policy
// when it is prepared to be displayed on chrome://policy.
- if (forced_disabled_sync_types_.Size() > 0) {
- entry->AddMessage(policy::PolicyMap::MessageType::kInfo,
- IDS_POLICY_BROWSING_DATA_DEPENDENCY_APPLIED_INFO,
- {base::UTF8ToUTF16(UserSelectableTypeSetToString(
- forced_disabled_sync_types_))});
- }
+ entry->AddMessage(policy::PolicyMap::MessageType::kInfo,
+ IDS_POLICY_BROWSING_DATA_DEPENDENCY_APPLIED_INFO,
+ {base::UTF8ToUTF16(UserSelectableTypeSetToString(
+ forced_disabled_sync_types_))});
}
+
} // namespace enterprise_idle
