chromium/src
0
Fork 0
Code Activity

Fix -Wloop-analysis: fuchsia/

Browse Source 
This has no effect on performance; it makes explicit a lifetime
extension of a temp that was already occurring.

Bug: 1223264
Change-Id: I99f6862d31ce32e0c3ce0ccdb60f759b8598db29
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3023551
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Auto-Submit: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#901003}
This commit is contained in:
Peter Kasting
2021-07-13 14:30:40 +00:00
committed by Chromium LUCI CQ
parent ad81f9782b
commit c1a0ce6f3f
1 changed files with 1 additions and 1 deletions

2
fuchsia/engine/web_instance_host/web_instance_host.cc

@ -373,7 +373,7 @@ bool MaybeAddCommandLineArgsFromConfig(const base::Value& config,
switches::kWebglMSAASampleCount,
};
for (const auto& arg : args->DictItems()) {
for (const auto arg : args->DictItems()) {
if (!base::Contains(kAllowedArgs, arg.first)) {
// TODO(https://crbug.com/1032439): Increase severity and return false
// once we have a mechanism for soft transitions of supported arguments.