0

Improve AutoPipInfoToString readability

Bug: 368058093, 386193409
Change-Id: Idcdc2848b6f0ce5772ccad784ac59c0ea0d16e85
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6440045
Reviewed-by: Frank Liberato <liberato@chromium.org>
Commit-Queue: Benjamin Keen <bkeen@google.com>
Cr-Commit-Position: refs/heads/main@{#1443848}
This commit is contained in:
Benjamin Keen
2025-04-07 17:12:45 -07:00
committed by Chromium LUCI CQ
parent 85c1c601a8
commit ff95bba00e

@ -34,17 +34,18 @@ std::string PictureInPictureEventsInfo::AutoPipReasonToString(
// static
std::string PictureInPictureEventsInfo::AutoPipInfoToString(
AutoPipInfo auto_pip_info) {
constexpr std::array<std::string, 2> bool_to_string{"false", "true"};
return base::StringPrintf(
"{Reason: %s, has audio focus: %s, is_playing: %s, was recently audible: "
"%s, has safe url: %s, meets media engagement conditions: %s, blocked "
"due to content setting: %s}",
AutoPipReasonToString(auto_pip_info.auto_pip_reason),
auto_pip_info.has_audio_focus ? "true" : "false",
auto_pip_info.is_playing ? "true" : "false",
auto_pip_info.was_recently_audible ? "true" : "false",
auto_pip_info.has_safe_url ? "true" : "false",
auto_pip_info.meets_media_engagement_conditions ? "true" : "false",
auto_pip_info.blocked_due_to_content_setting ? "true" : "false");
bool_to_string[auto_pip_info.has_audio_focus],
bool_to_string[auto_pip_info.is_playing],
bool_to_string[auto_pip_info.was_recently_audible],
bool_to_string[auto_pip_info.has_safe_url],
bool_to_string[auto_pip_info.meets_media_engagement_conditions],
bool_to_string[auto_pip_info.blocked_due_to_content_setting]);
}
} // namespace media