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

Unified Diff: mojo/public/cpp/bindings/lib/handle_interface_serialization.h

Issue 2449953008: Port messages sent by WebIDBDatabaseImpl to Mojo. (Closed)
Patch Set: Address more comments from dcheng@. Created 4 years, 1 month 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 | « content/test/BUILD.gn ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/public/cpp/bindings/lib/handle_interface_serialization.h
diff --git a/mojo/public/cpp/bindings/lib/handle_interface_serialization.h b/mojo/public/cpp/bindings/lib/handle_interface_serialization.h
index ecfb5bb5fedc476866d477ca5f675034c90d434a..8b26d84b64ba5657210b4e5b9fcf6bfbc5eef538 100644
--- a/mojo/public/cpp/bindings/lib/handle_interface_serialization.h
+++ b/mojo/public/cpp/bindings/lib/handle_interface_serialization.h
@@ -30,8 +30,10 @@ struct Serializer<AssociatedInterfacePtrInfoDataView<Base>,
AssociatedInterface_Data* output,
SerializationContext* context) {
DCHECK(!input.handle().is_valid() || !input.handle().is_local());
- DCHECK_EQ(input.handle().group_controller(),
- context->group_controller.get());
+ if (input.handle().is_valid()) {
+ DCHECK_EQ(input.handle().group_controller(),
+ context->group_controller.get());
+ }
output->version = input.version();
output->interface_id = input.PassHandle().release();
}
@@ -55,8 +57,10 @@ struct Serializer<AssociatedInterfaceRequestDataView<Base>,
AssociatedInterfaceRequest_Data* output,
SerializationContext* context) {
DCHECK(!input.handle().is_valid() || !input.handle().is_local());
- DCHECK_EQ(input.handle().group_controller(),
- context->group_controller.get());
+ if (input.handle().is_valid()) {
+ DCHECK_EQ(input.handle().group_controller(),
+ context->group_controller.get());
+ }
output->interface_id = input.PassHandle().release();
}
« no previous file with comments | « content/test/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698