Chromium Code Reviews| Index: components/sessions/BUILD.gn |
| diff --git a/components/sessions/BUILD.gn b/components/sessions/BUILD.gn |
| index f2ada0545aee1890d319f722324ae5c0f51b0e5d..a0523987927104015cdccb33c6d715f1c404c2cf 100644 |
| --- a/components/sessions/BUILD.gn |
| +++ b/components/sessions/BUILD.gn |
| @@ -6,14 +6,8 @@ if (is_android) { |
| import("//build/config/android/config.gni") |
| } |
| -component("sessions") { |
| +source_set("sessions_core") { |
|
blundell
2014/10/24 14:10:42
I would have an explanation here also of why this
|
| sources = [ |
| - # TODO(rohitrao): Split this target into three separate core, content, |
| - # and web targets. |
| - "content/content_serialized_navigation_builder.cc", |
| - "content/content_serialized_navigation_builder.h", |
| - "content/content_serialized_navigation_driver.cc", |
| - "content/content_serialized_navigation_driver.h", |
| "core/serialized_navigation_driver.h", |
| "serialized_navigation_entry.cc", |
| "serialized_navigation_entry.h", |
| @@ -25,8 +19,6 @@ component("sessions") { |
| deps = [ |
| "//base", |
| - "//base/third_party/dynamic_annotations", |
| - "//content/public/browser", |
| "//skia", |
| "//ui/base", |
| "//url", |
| @@ -37,6 +29,26 @@ component("sessions") { |
| } |
| } |
| +component("sessions_content") { |
| + sources = [ |
| + "content/content_serialized_navigation_builder.cc", |
| + "content/content_serialized_navigation_builder.h", |
| + "content/content_serialized_navigation_driver.cc", |
| + "content/content_serialized_navigation_driver.h", |
| + ] |
| + |
| + defines = [ "SESSIONS_IMPLEMENTATION" ] |
| + |
| + deps = [ |
| + ":sessions_core", |
| + "//base", |
| + "//base/third_party/dynamic_annotations", |
| + "//content/public/browser", |
| + "//ui/base", |
| + "//url", |
| + ] |
| +} |
| + |
| static_library("test_support") { |
| testonly = true |
| sources = [ |