Move session into //blimp/client/app and update GN files.

blimp/client/core/context/BUILD.gn

 # Copyright 2016 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.
 
 if (is_android) {
   import("//build/config/android/config.gni")
   import("//build/config/android/rules.gni")
 }
 
 source_set("context") {
+  visibility = [
+    ":unit_tests",
+    "//blimp/client/app:session",  # TODO(nyquist): Remove this. See crbug/651964.
+    "//blimp/client/core",
+    "//blimp/engine:browser_tests",

[David Trainor- moved to gerrit, 2016/10/07 06:36:48]

Is this blocked on porting over the final few unit tests to 0.6?  Should we tie
this to a bug for that?  Same with contents/ and other core targets that expose
browser_tests.  wdyt?

[nyquist, 2016/10/07 07:47:23]

Filed a new bug for this, and added a comment for all the browser_tests target
visibilities.

+  ]
+
   sources = [
     "blimp_client_context_impl.cc",
     "blimp_client_context_impl.h",
   ]
 
   public_deps = [
     "//base",
     "//blimp/client/core/compositor:compositor",
     "//blimp/client/core/contents",
     "//blimp/client/core/session",
@@ skipping 43 matching lines @@
       "android/dummy_blimp_client_context_android.cc",
       "android/dummy_blimp_client_context_android.h",
       "android/dummy_blimp_jni_registrar.cc",
     ]
 
     deps += [ ":dummy_context_jni_headers" ]
   }
 }
 
 source_set("unit_tests") {
+  visibility = [ "//blimp/client/core:unit_tests" ]
+
   testonly = true
 
   sources = [
     "blimp_client_context_impl_unittest.cc",
   ]
 
   deps = [
     ":context",
     "//blimp/client/public:public_headers",
     "//blimp/client/test",
     "//testing/gtest",
   ]
 }
 
 if (is_android) {
   android_library("context_java") {
-    visibility = [ "//blimp/client/*" ]
+    visibility = [ "//blimp/client/core/*" ]
 
     java_files = [ "android/java/src/org/chromium/blimp/core/BlimpClientContextImpl.java" ]
 
     deps = [
       "//base:base_java",
       "//blimp/client/core/common:common_java",
       "//blimp/client/core/contents:contents_java",
       "//blimp/client/core/feedback:feedback_java",
       "//blimp/client/core/settings:settings_java",
       "//blimp/client/core/switches:switches_java",
       "//blimp/client/public:public_headers_java",
       "//ui/android:ui_java",
     ]
   }
 
   android_library("dummy_context_java") {
-    visibility = [ "//blimp/client/*" ]
+    visibility = [ "//blimp/client/core/*" ]
 
     java_files = [ "android/java/src/org/chromium/blimp/core/DummyBlimpClientContext.java" ]
 
     deps = [
       "//base:base_java",
       "//blimp/client/public:public_headers_java",
       "//ui/android:ui_java",
     ]
   }
 
@@ skipping 10 matching lines @@
   generate_jni("dummy_context_jni_headers") {
     visibility = [ ":*" ]
 
     sources = [
       "android/java/src/org/chromium/blimp/core/DummyBlimpClientContext.java",
     ]
 
     jni_package = "blimp/client/core"
   }
 }