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

Unified Diff: blimp/engine/BUILD.gn

Issue 1985863002: Incorporate BlobChannel into Blimp image encode/decode pipeline. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@blobchannel-helium
Patch Set: Decoupled BlobChannelSenderHost from mojom::BlobChannel Created 4 years, 7 months 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/engine/BUILD.gn
diff --git a/blimp/engine/BUILD.gn b/blimp/engine/BUILD.gn
index 44b8d9ae9258bb20b0b9bf6a2acdb29767effc44..18f5bc782453da1000e3a45c67d203a9523adcc4 100644
--- a/blimp/engine/BUILD.gn
+++ b/blimp/engine/BUILD.gn
@@ -62,7 +62,7 @@ source_set("app") {
":app_settings",
":app_switches",
":app_ui",
- ":blob_channel",
+ ":blob_channel_service",
":common",
":crash",
":renderer",
@@ -252,6 +252,9 @@ 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. Al
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",
]
@@ -299,7 +302,7 @@ source_set("session") {
}
# 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",
@@ -375,6 +378,8 @@ source_set("feature_unit_tests") {
"//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",

Powered by Google App Engine
This is Rietveld 408576698