Tied up BlimpNavigationController to NavigationFeature

blimp/client/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/ui.gni")
 
 if (is_android) {
   import("//build/config/android/config.gni")
   import("//build/config/android/rules.gni")
 }
@@ skipping 152 matching lines @@
     "//blimp/common/proto",
   ]
 }
 
 source_set("test_support") {
   testonly = true
 
   sources = [
     "feature/mock_ime_feature_delegate.cc",
     "feature/mock_ime_feature_delegate.h",
+    "feature/mock_navigation_feature.cc",

[nyquist, 2016/07/25 22:59:06]

Could we move these two new files to //blimp/client/core/contents?

+    "feature/mock_navigation_feature.h",
     "feature/mock_navigation_feature_delegate.cc",
     "feature/mock_navigation_feature_delegate.h",
     "feature/mock_render_widget_feature_delegate.cc",
     "feature/mock_render_widget_feature_delegate.h",
     "session/test_client_session.cc",
     "session/test_client_session.h",
   ]
 
   deps = [
     ":session",
     "//skia",
+    "//url",
+  ]
+
+  public_deps = [
     "//testing/gmock",
-    "//url",
   ]
 }
 
 source_set("compositor") {
   sources = [
     "feature/compositor/blimp_client_picture_cache.cc",
     "feature/compositor/blimp_client_picture_cache.h",
     "feature/compositor/blimp_compositor.cc",
     "feature/compositor/blimp_compositor.h",
     "feature/compositor/blimp_compositor_manager.cc",
@@ skipping 341 matching lines @@
   instrumentation_test_apk("blimp_test_apk") {
     apk_name = "BlimpTest"
     apk_under_test = ":blimp_apk"
     android_manifest = blimp_test_apk_manifest
     android_manifest_dep = ":blimp_test_apk_manifest"
     deps = [
       ":blimp_test_java",
     ]
   }
 }