Adds GeolocationFeature for Blimp Geolocation project.

blimp/client/BUILD.gn

Index: blimp/client/BUILD.gn
diff --git a/blimp/client/BUILD.gn b/blimp/client/BUILD.gn
index 8c536ea8b5ecca4f9354073b3ee1d7d2e412d1b0..10b13fdb4ca7d982197da0e2b0e01a48981fec71 100644
--- a/blimp/client/BUILD.gn
+++ b/blimp/client/BUILD.gn
@@ -93,6 +93,8 @@ 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",
@@ -108,6 +110,7 @@ source_set("feature") {
     "//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",
@@ -133,6 +136,7 @@ source_set("test_support") {
 
   deps = [
     ":session",
+    "//device/geolocation:device_geolocation",
     "//skia",
     "//testing/gmock",
     "//url",
@@ -198,6 +202,7 @@ source_set("feature_unit_tests") {
   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",
   ]
 
@@ -213,6 +218,8 @@ source_set("feature_unit_tests") {
     "//blimp/net:test_support",
     "//blimp/test:support",
     "//cc/proto",
+    "//device/geolocation:device_geolocation",
+    "//device/geolocation:unittests",
     "//net",
     "//net:test_support",
     "//skia",