Implementation and fixes for Blimp client/engine E2E communication.

blimp/BUILD.gn

 # Copyright 2015 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.
 
 import("//testing/test.gni")
 
 # Convenience meta-target for all of Blimp's production & test code.
 group("blimp") {
   # In spite of the name, this really just means that non-test targets
   # cannot depend on this one, and that it can depend on test targets,
   # which fits how we intend to use it.
   testonly = true
 
   deps = [
     ":blimp_tests",
     "//blimp/client:blimp_client",
     "//blimp/common:blimp_common",
   ]
 
   if (is_android) {
     deps += [ "//blimp/client:blimp_apk" ]
   }
 
   if (is_linux) {
     deps += [
+      "//blimp/client:blimp_shell",

[haibinlu, 2016/01/04 19:45:45]

remove this?

[Kevin M, 2016/01/04 20:42:12]

It's important to keep this in, so that the Linux client is built by the
waterfall.

[Kevin M, 2016/01/04 21:09:10]

Talked about this offline. Didn't see it declared on line 32; removed
incorrect/redundant entry.

       "//blimp/engine:blimp_engine",
       "//blimp/engine:blimp_engine_bundle",
     ]
   }
 
   if (is_linux && !is_chromeos) {
     deps += [ "//blimp/client:blimp_shell" ]
   }
 }
 
@@ skipping 23 matching lines @@
 }
 
 # TODO(GYP): Delete this after we've converted everything to GN.
 # The _run targets exist only for compatibility w/ GYP.
 group("blimp_unittests_run") {
   testonly = true
   deps = [
     ":blimp_unittests",
   ]
 }