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

Unified Diff: blimp/client/core/contents/BUILD.gn

Issue 2058263002: Tied up BlimpNavigationController to NavigationFeature (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@blimp_core
Patch Set: Removed BlimpContentsTest.java Created 4 years, 4 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/client/core/contents/BUILD.gn
diff --git a/blimp/client/core/contents/BUILD.gn b/blimp/client/core/contents/BUILD.gn
index 33a5b40286d99a624cea7cbccb04ca832d5b4e2c..c3f8ab8b0fcb2d0aa6976da160460dd399fc9fdd 100644
--- a/blimp/client/core/contents/BUILD.gn
+++ b/blimp/client/core/contents/BUILD.gn
@@ -16,14 +16,23 @@ source_set("contents") {
"blimp_navigation_controller_delegate.h",
"blimp_navigation_controller_impl.cc",
"blimp_navigation_controller_impl.h",
+ "navigation_feature.cc",
+ "navigation_feature.h",
]
public_deps = [
"//base",
"//blimp/client/public:public_headers",
+ "//blimp/net",
+ "//net:net",
"//url",
]
+ deps = [
+ "//blimp/common",
+ "//components/url_formatter",
+ ]
+
if (is_android) {
sources += [
"android/blimp_contents_factory.cc",
@@ -38,12 +47,32 @@ source_set("contents") {
"android/blimp_navigation_controller_impl_android.h",
]
- deps = [
- ":jni_headers",
- ]
+ deps += [ ":jni_headers" ]
}
}
+source_set("test_support") {
+ testonly = true
+
+ sources = [
+ "fake_navigation_feature.cc",
+ "fake_navigation_feature.h",
+ "mock_navigation_feature_delegate.cc",
+ "mock_navigation_feature_delegate.h",
+ ]
+
+ deps = [
+ "//base",
+ "//skia",
+ "//url",
+ ]
+
+ public_deps = [
+ "//blimp/client/core/contents",
+ "//testing/gmock",
+ ]
+}
+
source_set("unit_tests") {
visibility = [ "//blimp/client/core:unit_tests" ]
@@ -52,12 +81,17 @@ source_set("unit_tests") {
sources = [
"blimp_contents_impl_unittest.cc",
"blimp_navigation_controller_impl_unittest.cc",
+ "navigation_feature_unittest.cc",
]
deps = [
":contents",
+ ":test_support",
"//base",
"//blimp/client/public:public_headers",
+ "//blimp/common",
+ "//blimp/net:test_support",
+ "//net:test_support",
"//testing/gtest",
"//url",
]
« no previous file with comments | « blimp/client/app/linux/blimp_main.cc ('k') | blimp/client/core/contents/android/blimp_contents_observer_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698