Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(992)

Unified Diff: ipc/ipc_message_utils.cc

Issue 2689673002: Inline base::BinaryValue into base::Value (Closed)
Patch Set: Rebase Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « extensions/renderer/argument_spec_unittest.cc ('k') | media/base/video_frame_metadata.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ipc/ipc_message_utils.cc
diff --git a/ipc/ipc_message_utils.cc b/ipc/ipc_message_utils.cc
index fb45218bff3da682a4af16a16b0a62d5ac4c637a..e7d8747ff5767dc054e2c7f84117716a6a5567a0 100644
--- a/ipc/ipc_message_utils.cc
+++ b/ipc/ipc_message_utils.cc
@@ -110,9 +110,7 @@ void GetValueSize(base::PickleSizer* sizer,
break;
}
case base::Value::Type::BINARY: {
- const base::BinaryValue* binary =
- static_cast<const base::BinaryValue*>(value);
- sizer->AddData(static_cast<int>(binary->GetSize()));
+ sizer->AddData(static_cast<int>(value->GetSize()));
break;
}
case base::Value::Type::DICTIONARY: {
@@ -180,9 +178,7 @@ void WriteValue(base::Pickle* m, const base::Value* value, int recursion) {
break;
}
case base::Value::Type::BINARY: {
- const base::BinaryValue* binary =
- static_cast<const base::BinaryValue*>(value);
- m->WriteData(binary->GetBuffer(), static_cast<int>(binary->GetSize()));
+ m->WriteData(value->GetBuffer(), static_cast<int>(value->GetSize()));
break;
}
case base::Value::Type::DICTIONARY: {
« no previous file with comments | « extensions/renderer/argument_spec_unittest.cc ('k') | media/base/video_frame_metadata.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698