Adds EngineGeolocationFeature for Blimp Geolocation project.

blimp/engine/feature/geolocation/blimp_location_provider.cc

Index: blimp/engine/feature/geolocation/blimp_location_provider.cc
diff --git a/blimp/engine/feature/geolocation/blimp_location_provider.cc b/blimp/engine/feature/geolocation/blimp_location_provider.cc
index 23a9a7f0e51b39b2c799e6242e6dc421ed3566da..d42368b1d3fc4a31d8b0d7042f594f632f120c7e 100644
--- a/blimp/engine/feature/geolocation/blimp_location_provider.cc
+++ b/blimp/engine/feature/geolocation/blimp_location_provider.cc
@@ -41,17 +41,25 @@ void BlimpLocationProvider::OnPermissionGranted() {
   NOTIMPLEMENTED();
 }
 
-void BlimpLocationProvider::NotifyCallback(
+void BlimpLocationProvider::OnLocationResponse(
     const content::Geoposition& position) {
-  DCHECK(!callback_.is_null());
+  NotifyCallback(position);

[Kevin M, 2016/06/29 17:52:55]

Pick one: NOTIMPLEMENTED() with nothing else, or add an implementation. ;)

[CJ, 2016/07/11 23:21:06]

Done.

+  NOTIMPLEMENTED();
+}
 
-  callback_.Run(this, position);
+void BlimpLocationProvider::SetDelegate(
+    BlimpLocationProvider::BlimpLocationProviderDelegate* delegate) {
+  DCHECK(delegate);
+  DCHECK(!delegate);

[Kevin M, 2016/06/29 17:52:55]

Not sure if you intended to do this. ;)

[CJ, 2016/07/11 23:21:06]

oops.

+
+  delegate_ = delegate;
 }
 
-void BlimpLocationProvider::OnLocationResponse(
+void BlimpLocationProvider::NotifyCallback(
     const content::Geoposition& position) {
-  NotifyCallback(position);
-  NOTIMPLEMENTED();
+  DCHECK(!callback_.is_null());
+
+  callback_.Run(this, position);
 }
 
 void BlimpLocationProvider::SetUpdateCallback(