chromium/src
0
Fork 0
Code Activity

CodeHealth: Remove use of Value::GetAsString.

Browse Source 
This change removes the use of Value::GetAsString and replaces with
Value::is_string, Value::GetString(), Value::GetIfString().

This CL was uploaded by git cl split.

R=rockot@google.com

Bug: 1187007
Change-Id: Ic8330f6215683f3d54683d25fc6cde6da4cb0c9c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3098906
Auto-Submit: Sammie Quon <sammiequon@chromium.org>
Reviewed-by: Ken Rockot <rockot@google.com>
Commit-Queue: Ken Rockot <rockot@google.com>
Cr-Commit-Position: refs/heads/master@{#912868}
This commit is contained in:
Sammie Quon
2021-08-18 02:34:48 +00:00
committed by Chromium LUCI CQ
parent 387d6dcf9a
commit dab84ebae9
1 changed files with 3 additions and 3 deletions

6
ipc/ipc_message_utils.cc

@ -111,10 +111,10 @@ void WriteValue(base::Pickle* m, const base::Value* value, int recursion) {
break;
}
case base::Value::Type::STRING: {
std::string val;
result = value->GetAsString(&val);
const std::string* val = value->GetIfString();
result = !!val;
DCHECK(result);
WriteParam(m, val);
WriteParam(m, *val);
break;
}
case base::Value::Type::BINARY: {