Commit 7d5214a6 authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Chromium LUCI CQ

Fix potential use-after-move in //c/b/android/usage_stats/

Fix use-after-move (potential) bugs found by the
"bugprone-use-after-move" clang-tidy check.

Bug: 1122844
Change-Id: I34a773f1610ff151708534e644b5ddda4d27f9b2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2584964
Commit-Queue: Patrick Noland <pnoland@chromium.org>
Auto-Submit: Maksim Ivanov <emaxx@chromium.org>
Reviewed-by: default avatarPatrick Noland <pnoland@chromium.org>
Cr-Commit-Position: refs/heads/master@{#835922}
parent fd08e255
...@@ -99,9 +99,9 @@ static void JNI_NotificationSuspender_StoreNotificationResources( ...@@ -99,9 +99,9 @@ static void JNI_NotificationSuspender_StoreNotificationResources(
GURL origin(std::move(origin_strings[i])); GURL origin(std::move(origin_strings[i]));
if (!origin.is_valid() || !origin.SchemeIsHTTPOrHTTPS()) if (!origin.is_valid() || !origin.SchemeIsHTTPOrHTTPS())
continue; continue;
resources_by_context[GetContext(profile, origin)].emplace_back( PlatformNotificationContext* context = GetContext(profile, origin);
NotificationResourceData{std::move(id_strings[i]), std::move(origin), resources_by_context[context].emplace_back(NotificationResourceData{
std::move(resources[i])}); std::move(id_strings[i]), std::move(origin), std::move(resources[i])});
} }
// Store resources in each context. // Store resources in each context.
......
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