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

Unified Diff: mojo/public/cpp/bindings/lib/string_serialization.cc

Issue 1520153002: [mojo] Allow value deserialization to fail (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@bindings-3-misc-support
Patch Set: merge Created 5 years 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
Index: mojo/public/cpp/bindings/lib/string_serialization.cc
diff --git a/mojo/public/cpp/bindings/lib/string_serialization.cc b/mojo/public/cpp/bindings/lib/string_serialization.cc
index be1508d425ceceadfa3e5eb70ff7f0ca182531a8..936493edffbd9bb6d495cb54ffaccbdf69fc57ee 100644
--- a/mojo/public/cpp/bindings/lib/string_serialization.cc
+++ b/mojo/public/cpp/bindings/lib/string_serialization.cc
@@ -28,7 +28,7 @@ void Serialize_(const String& input,
}
}
-void Deserialize_(internal::String_Data* input,
+bool Deserialize_(internal::String_Data* input,
String* output,
internal::SerializationContext* context) {
if (input) {
@@ -37,6 +37,7 @@ void Deserialize_(internal::String_Data* input,
} else {
output->reset();
}
+ return true;
}
} // namespace mojo

Powered by Google App Engine
This is Rietveld 408576698