Commit abced152 authored by Dominic Battre's avatar Dominic Battre Committed by Commit Bot

Annotate boolean parameter in function call

This is just a tiny cleanup that has been suggested on a previous CL.

Bug: 846694
Change-Id: I0462c6b6a88d434ccbd4f41fa11513628b0e7ea7
Reviewed-on: https://chromium-review.googlesource.com/1098662Reviewed-by: default avatarVaclav Brozek <vabr@chromium.org>
Commit-Queue: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#566780}
parent 482d82ff
......@@ -85,8 +85,10 @@ void ContentPasswordManagerDriver::FillPasswordForm(
void ContentPasswordManagerDriver::AllowPasswordGenerationForForm(
const autofill::PasswordForm& form) {
if (!GetPasswordGenerationManager()->IsGenerationEnabled(true))
if (!GetPasswordGenerationManager()->IsGenerationEnabled(
/*log_debug_data=*/true)) {
return;
}
GetPasswordGenerationAgent()->FormNotBlacklisted(form);
}
......
......@@ -42,8 +42,7 @@ void PasswordGenerationManager::ProcessPasswordRequirements(
const std::vector<autofill::FormStructure*>& forms) {
// IsGenerationEnabled is called multiple times and it is sufficient to
// log debug data once.
const bool kLogDebugData = false;
if (!IsGenerationEnabled(kLogDebugData))
if (!IsGenerationEnabled(/*log_debug_data=*/false))
return;
// It is legit to have no PasswordRequirementsService on some platforms where
......@@ -75,8 +74,7 @@ void PasswordGenerationManager::DetectFormsEligibleForGeneration(
const std::vector<autofill::FormStructure*>& forms) {
// IsGenerationEnabled is called multiple times and it is sufficient to
// log debug data once. This is it!
const bool kLogDebugData = true;
if (!IsGenerationEnabled(kLogDebugData))
if (!IsGenerationEnabled(/*log_debug_data=*/true))
return;
std::vector<autofill::PasswordFormGenerationData>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment