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

Unified Diff: blimp/common/proto/helium_service.proto

Issue 2462183002: GRPC Stream implementation of HeliumStream
Patch Set: Fixed a few minor comments 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
Index: blimp/common/proto/helium_service.proto
diff --git a/blimp/common/proto/helium_service.proto b/blimp/common/proto/helium_service.proto
new file mode 100644
index 0000000000000000000000000000000000000000..0eb4f802185631ead5050576fda2971b68ca086c
--- /dev/null
+++ b/blimp/common/proto/helium_service.proto
@@ -0,0 +1,24 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+syntax = "proto3";
+
+option optimize_for = LITE_RUNTIME;
+
+package blimp;
+
+// This message wraps around a serialized set of bytes from a
Garrett Casto 2016/11/11 00:15:53 Nit: You are line wrapping really early here.
+// generic message used above the transport layer. This is
+// intentionally opaque so that the transport layer does not
+// know the details of the exact type(s) of messages inside
+// this serialized bytes.
+message HeliumWrapper {
+ bytes serialized_helium_message = 1;
+}
+
+service HeliumService {
+ // A bi-directional streaming API that sends and receives a
+ // |HeliumWrapper| protobuf message.
+ rpc Stream(stream HeliumWrapper) returns (stream HeliumWrapper) {}
+}

Powered by Google App Engine
This is Rietveld 408576698