Commit acf33d8d authored by Dave Tapuska's avatar Dave Tapuska Committed by Commit Bot

Fix shadowed variables in the content layer.

Shadowed variables can make code harder to read. Don't support them
in the content layer.

BUG=794619

This CL was uploaded by git cl split.

R=dmurph@chromium.org

Change-Id: Ie36fb07ec35d59a1bfbeb21773fe68b04ac14bf1
Reviewed-on: https://chromium-review.googlesource.com/923352Reviewed-by: default avatarMarijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537390}
parent e2e00382
...@@ -939,13 +939,14 @@ void LocalStorageContextMojo::OnGotMetaData( ...@@ -939,13 +939,14 @@ void LocalStorageContextMojo::OnGotMetaData(
continue; continue;
} }
LocalStorageOriginMetaData data; LocalStorageOriginMetaData row_data;
if (!data.ParseFromArray(row->value.data(), row->value.size())) { if (!row_data.ParseFromArray(row->value.data(), row->value.size())) {
// TODO(mek): Deal with database corruption. // TODO(mek): Deal with database corruption.
continue; continue;
} }
info.data_size = data.size_bytes(); info.data_size = row_data.size_bytes();
info.last_modified = base::Time::FromInternalValue(data.last_modified()); info.last_modified =
base::Time::FromInternalValue(row_data.last_modified());
result.push_back(std::move(info)); result.push_back(std::move(info));
} }
// Add any origins for which LevelDBWrappers exist, but which haven't // Add any origins for which LevelDBWrappers exist, but which haven't
......
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