Incorporate BlobChannel into Blimp image encode/decode pipeline.

blimp/engine/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("//build/config/features.gni")
 import("//build/config/sanitizers/sanitizers.gni")
 import("//build/config/ui.gni")
 import("//mojo/public/tools/bindings/mojom.gni")
 import("//tools/grit/repack.gni")
 import("//tools/grit/grit_rule.gni")
@@ skipping 44 matching lines @@
     "app/blimp_content_main_delegate.h",
   ]
 
   deps = [
     ":app_config",
     ":app_net",
     ":app_permissions",
     ":app_settings",
     ":app_switches",
     ":app_ui",
-    ":blob_channel",
+    ":blob_channel_service",
     ":common",
     ":crash",
     ":renderer",
     ":session",
     "//base",
     "//blimp/common",
     "//blimp/common/proto",
     "//blimp/engine:blob_channel_mojo_cpp_sources",
     "//blimp/net",
     "//components/crash/content/app:app_breakpad_mac_win_to_be_deleted",
@@ skipping 169 matching lines @@
     "//ui/base/ime",
     "//ui/resources",
     "//ui/wm",
   ]
 }
 
 source_set("renderer") {
   sources = [
     "renderer/blimp_content_renderer_client.cc",
     "renderer/blimp_content_renderer_client.h",
+    "renderer/blimp_content_renderer_client.h",

[nyquist, 2016/05/24 00:05:41]

Nit: I don't think you need to list this twice. Also, interesting that our tools
don't inform us of this.

[Kevin M, 2016/05/27 22:35:30]

Done.

+    "renderer/blob_channel_sender_host.cc",
+    "renderer/blob_channel_sender_host.h",
     "renderer/engine_image_serialization_processor.cc",
     "renderer/engine_image_serialization_processor.h",
   ]
 
   deps = [
     ":blob_channel_mojo",
     "//base",
     "//blimp/common",
     "//blimp/common/proto",
     "//cc",
@@ skipping 27 matching lines @@
     "//content/public/browser",
     "//net",
     "//ui/aura",
     "//ui/base/ime",
     "//ui/gfx/geometry",
     "//ui/wm",
   ]
 }
 
 # Implements the browser portions of the Mojo bridge to BlobChannel.
-source_set("blob_channel") {
+source_set("blob_channel_service") {
   sources = [
     "mojo/blob_channel_service.cc",
     "mojo/blob_channel_service.h",
   ]
   deps = [
     ":blob_channel_mojo",
   ]
 }
 
 mojom("blob_channel_mojo") {
@@ skipping 55 matching lines @@
   deps = [
     ":feature",
     "//base",
     "//base/test:run_all_unittests",
     "//base/test:test_support",
     "//blimp/common",
     "//blimp/common/proto",
     "//blimp/engine:app_settings",
     "//blimp/net",
     "//blimp/net:test_support",
+    "//blimp/net:test_support",

[nyquist, 2016/05/24 00:05:41]

Nit: I don't think you need to list this twice.

[Kevin M, 2016/05/27 22:35:30]

Done.

+    "//content",
     "//content/public/browser",
     "//net",
     "//net:test_support",
     "//testing/gmock",
     "//testing/gtest",
     "//third_party/WebKit/public:blink_headers",
     "//ui/base/ime",
   ]
 }
 
@@ skipping 77 matching lines @@
       _rebased_dockerfile,
       "--startup-script",
       _rebased_startup_script,
       "--manifest",
       _rebased_manifest,
       "--output",
       rebase_path(_bundle),
     ]
   }
 }