Clean up geolocation code now that it's all upstream.

chrome/android/BUILD.gn

 # Copyright 2014 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/android/config.gni")
 import("//build/config/android/rules.gni")
 import("//build/module_args/v8.gni")
 import("//chrome/version.gni")
 import("//third_party/icu/config.gni")
 import("channel.gni")
@@ skipping 150 matching lines @@
     "java/ResourceId.template",
   ]
   package_name = "org/chromium/chrome/browser"
   inputs = [
     "../browser/android/resource_id.h",
   ]
 }
 
 # GYP: //chrome/chrome_shell.gypi:libchromeshell_base
 static_library("chrome_shell_base") {
-  sources = [
-    "shell/chrome_shell_google_location_settings_helper.cc",
-    "shell/chrome_shell_google_location_settings_helper.h",
-  ]
+  #sources = [

[Michael van Ouwerkerk, 2015/01/19 12:03:14]

Did you mean to delete this section?

[newt (away), 2015/01/20 16:58:51]

Yes, though I'm wondering if I can/should delete this entire build target. After
removing these sources, the target is only used for its list of deps... so it
probably shouldn't be a "static_library" target anymore. Is it OK to have a
target that's just a list of deps and doesn't actually do anything? Or should I
just copy the deps into the two targets that currently depend on
chrome_shell_base?

+  #  "shell/chrome_shell_google_location_settings_helper.cc",
+  #  "shell/chrome_shell_google_location_settings_helper.h",
+  #]
 
   deps = [
     "//chrome/browser/ui",
     "//chrome:chrome_android_core",
     "//base",
     "//base/allocator",
     "//content/public/app:browser",
     "//skia",
   ]
 
@@ skipping 324 matching lines @@
   ]
 }
 
 zip("chrome_version_srcjar") {
   inputs = [
     chrome_version_java_file,
   ]
   output = "$target_gen_dir/$target_name.srcjar"
   base_dir = chrome_version_java_dir
 }