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

blimp/client/core/context/BUILD.gn

Index: blimp/client/core/context/BUILD.gn
diff --git a/blimp/client/core/context/BUILD.gn b/blimp/client/core/context/BUILD.gn
index 5b767d7b68f6fc1656503659b6920a3cf3c8da97..9e99f24f4a15eb03a0cf14f8286239b374689d35 100644
--- a/blimp/client/core/context/BUILD.gn
+++ b/blimp/client/core/context/BUILD.gn
@@ -8,6 +8,13 @@ if (is_android) {
 }
 
 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",
@@ -71,6 +78,8 @@ source_set("dummy") {
 }
 
 source_set("unit_tests") {
+  visibility = [ "//blimp/client/core:unit_tests" ]
+
   testonly = true
 
   sources = [
@@ -87,7 +96,7 @@ source_set("unit_tests") {
 
 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" ]
 
@@ -104,7 +113,7 @@ if (is_android) {
   }
 
   android_library("dummy_context_java") {
-    visibility = [ "//blimp/client/*" ]
+    visibility = [ "//blimp/client/core/*" ]
 
     java_files = [ "android/java/src/org/chromium/blimp/core/DummyBlimpClientContext.java" ]