Pass through geofencing API calls to the content layer.

Source/modules/geofencing/CircularRegion.cpp

 // 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.
 
 #include "config.h"
 #include "modules/geofencing/CircularRegion.h"
 
 #include "bindings/core/v8/Dictionary.h"
+#include "public/platform/WebCircularRegion.h"
+#include "public/platform/WebString.h"
 
 namespace blink {
 
 CircularRegionInit::CircularRegionInit(const Dictionary& init)
     : latitude(0)
     , longitude(0)
     , radius(0)
 {
     DictionaryHelper::get(init, "id", id);
     DictionaryHelper::get(init, "latitude", latitude);
     DictionaryHelper::get(init, "longitude", longitude);
     DictionaryHelper::get(init, "radius", radius);
 }
 
 CircularRegion* CircularRegion::create(const Dictionary& init)
 {
-    return new CircularRegion(CircularRegionInit(init));
+    CircularRegionInit rinit(init);

[abarth-chromium, 2014/08/23 05:43:34]

rinit -> please use complete words in variable names.

[Marijn Kruisselbrink, 2014/08/25 22:25:25]

Done.

+    return new CircularRegion(WebCircularRegion::create(rinit.id, rinit.latitude, rinit.longitude, rinit.radius));

[abarth-chromium, 2014/08/23 05:43:34]

So much mallocing.  Can't WebCircularRegion just be a struct that contains its
data?

 }
 
-CircularRegion::CircularRegion(const CircularRegionInit& init)
-    : GeofencingRegion(init.id)
-    , m_latitude(init.latitude)
-    , m_longitude(init.longitude)
-    , m_radius(init.radius)
+CircularRegion::CircularRegion(const WebCircularRegion& region)
+    : GeofencingRegion(region)
 {
     ScriptWrappable::init(this);
 }
 
+double CircularRegion::latitude() const
+{
+    return webRegion().toConst<WebCircularRegion>().latitude();

[abarth-chromium, 2014/08/23 05:43:34]

So ugly...

Why not just have WebCircularRegion be a struct and have CircularRegion hold a
pointer to that struct?

+}
+
+double CircularRegion::longitude() const
+{
+    return webRegion().toConst<WebCircularRegion>().longitude();
+}
+
+double CircularRegion::radius() const
+{
+    return webRegion().toConst<WebCircularRegion>().radius();
+}
+
 } // namespace blink