Adds GeolocationFeature for Blimp Geolocation project.

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 75 matching lines @@
     "//base/test:test_support",
     "//testing/gmock",
     "//testing/gtest",
   ]
 }
 
 source_set("feature") {
   sources = [
     "core/contents/tab_control_feature.cc",
     "core/contents/tab_control_feature.h",
+    "feature/geolocation_feature.cc",
+    "feature/geolocation_feature.h",
     "feature/ime_feature.cc",
     "feature/ime_feature.h",
     "feature/render_widget_feature.cc",
     "feature/render_widget_feature.h",
     "feature/settings_feature.cc",
     "feature/settings_feature.h",
   ]
 
   deps = [
     "//base:base",
     "//blimp/common",
     "//blimp/net",
     "//cc",
     "//cc/proto",
     "//components/url_formatter",
+    "//device/geolocation:device_geolocation",

[Kevin M, 2016/08/15 20:35:40]

Hm, this target should probably be named "geolocation", which becomes the more
succinct target "//device/geolocation". Not really in your CL scope, but just
pointing this out as another point for potential cleanup.

[CJ, 2016/08/15 21:59:31]

yeah I agree. Not sure why it was made like that. 

     "//net:net",
     "//skia",
     "//ui/base/ime:text_input_types",
     "//ui/gfx/geometry:geometry",
   ]
 
   public_deps = [
     "//blimp/common/proto",
   ]
 }
 
 source_set("test_support") {
   testonly = true
 
   sources = [
     "feature/mock_ime_feature_delegate.cc",
     "feature/mock_ime_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",
+    "//device/geolocation:device_geolocation",
     "//skia",
     "//testing/gmock",
     "//url",
   ]
 }
 
 source_set("compositor") {
   sources = [
     "feature/compositor/blimp_compositor.cc",
     "feature/compositor/blimp_compositor.h",
@@ skipping 45 matching lines @@
     "//skia",
   ]
 }
 
 source_set("feature_unit_tests") {
   testonly = true
 
   sources = [
     "core/contents/tab_control_feature_unittest.cc",
     "feature/compositor/blimp_compositor_manager_unittest.cc",
+    "feature/geolocation_feature_unittest.cc",
     "feature/render_widget_feature_unittest.cc",
   ]
 
   deps = [
     ":compositor",
     ":test_support",
     "//base",
     "//base/test:test_support",
     "//blimp/client/core/compositor",
     "//blimp/common",
     "//blimp/common/proto",
     "//blimp/net",
     "//blimp/net:test_support",
     "//blimp/test:support",
     "//cc/proto",
+    "//device/geolocation:device_geolocation",
+    "//device/geolocation:unittests",
     "//net",
     "//net:test_support",
     "//skia",
     "//testing/gmock",
     "//testing/gtest",
     "//ui/events:gesture_detection",
     "//ui/gfx/geometry",
   ]
 
   if (is_linux) {
@@ skipping 255 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",
     ]
   }
 }