Geolocation: move mojom from WebKit to device/geolocation/public/interfaces

device/geolocation/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.
 
 import("//build/config/features.gni")
 
 if (is_android) {
   import("//build/config/android/rules.gni")  # For generate_jni().
 }
 
 component("device_geolocation") {

[ortuno, 2016/08/02 14:55:45]

Shouldn't this be split into two separate targets? One for the service and one
for everything else. There are users of device/geolocation that don't need the
service or the bindings e.g. autofill.

[Ken Rockot(use gerrit already), 2016/08/02 16:23:33]

Maybe, but it's not clear how much that matters. I think eventually anyone
consuming geolocation would be doing so through service interfaces. In that case
there would certainly be a target split, but it'd be to avoid dependencies on
the impl rather than avoiding them on the interfaces.

   defines = [ "DEVICE_GEOLOCATION_IMPLEMENTATION" ]
 
   sources = [
     "access_token_store.h",
     "android/geolocation_jni_registrar.cc",
     "android/geolocation_jni_registrar.h",
     "empty_wifi_data_provider.cc",
     "empty_wifi_data_provider.h",
     "geolocation_delegate.cc",
     "geolocation_delegate.h",
@@ skipping 42 matching lines @@
     "wifi_data_provider_win.h",
     "wifi_polling_policy.h",
   ]
 
   deps = [
     "//base",
     "//google_apis:google_apis",
     "//mojo/edk/system",
     "//mojo/public/cpp/bindings",
     "//net",
-
-    # TODO(mcasas): move geolocation.mojom to public/interfaces.
-    "//third_party/WebKit/public:mojo_bindings",
     "//ui/gfx",
   ]
-
+  public_deps = [
+    "//device/geolocation/public/interfaces:geolocation_mojom",
+  ]
   if (is_android) {
     sources -= [
       "network_location_provider.cc",
       "network_location_provider.h",
       "network_location_request.cc",
       "network_location_request.h",
     ]
     deps += [ ":geolocation_jni_headers" ]
   }
 
@@ skipping 101 matching lines @@
   }
 
   if (is_android) {
     sources -= [
       "network_location_provider_unittest.cc",
       "wifi_data_provider_common_unittest.cc",
     ]
     deps += [ ":geolocation_java_test_support" ]
   }
 }