GRPC Stream implementation of HeliumStream

blimp/common/proto/BUILD.gn

Index: blimp/common/proto/BUILD.gn
diff --git a/blimp/common/proto/BUILD.gn b/blimp/common/proto/BUILD.gn
index c75c5b981a6fa763be725b33be793f23d1bf8611..6538ac1281c1df545249def929568b75a8c92316 100644
--- a/blimp/common/proto/BUILD.gn
+++ b/blimp/common/proto/BUILD.gn
@@ -1,4 +1,4 @@
-# Copyright 2015 The Chromium Authors. All rights reserved.
+# 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.
 
@@ -7,6 +7,7 @@ import("//third_party/protobuf/proto_library.gni")
 group("proto") {
   public_deps = [
     ":blimp_proto",
+    ":helium_service",
     "//third_party/protobuf:protobuf_lite",
   ]
 }
@@ -16,10 +17,24 @@ component("blimp_proto") {
   visibility = [ ":proto" ]
 
   public_deps = [
+    ":helium_service",
     ":proto_internal",
   ]
 }
 
+proto_library("helium_service") {

[Kevin M, 2016/10/31 21:33:25]

What about putting this inside //blimp/helium/service ?

[perumaal, 2016/10/31 22:11:01]

I think there is a bit of code consolidation here *after* the grpc stuff is
checked in. However, I do not think that this one proto file necessitates a new
folder though.

+  sources = [
+    "helium_service.proto",
+  ]
+
+  generator_plugin_label = "//third_party/grpc:grpc_cpp_plugin"
+  generator_plugin_suffix = ".grpc.pb"
+
+  deps = [
+    "//third_party/grpc:grpc++_unsecure",
+  ]
+}
+
 proto_library("proto_internal") {
   # Only expose the target to the "blimp_proto" target.
   visibility = [ ":blimp_proto" ]