Pass through the geofencing API calls to the content layer.

public/platform/WebGeofencingRegion.h

+// 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.
+
+#ifndef WebGeofencingRegion_h
+#define WebGeofencingRegion_h
+
+#include "public/platform/WebString.h"
+
+namespace WebCore { class GeofencingRegion; }
+
+namespace blink {
+
+class WebCircularRegion;
+
+class WebGeofencingRegion {
+public:
+    WebGeofencingRegion() { }
+    WebGeofencingRegion(const WebGeofencingRegion& other) { assign(other); }
+    WebGeofencingRegion& operator=(const WebGeofencingRegion& other)
+    {
+        assign(other);
+        return *this;
+    }
+    ~WebGeofencingRegion() { reset(); }

[Peter Beverloo, 2014/07/14 17:23:09]

Mark this as virtual and have empty destructors in the classes inheriting from
this, just to err on the safe side (if any of them ever have non-scalar
members)?

+
+    BLINK_EXPORT void assign(const WebGeofencingRegion&);
+    BLINK_EXPORT void reset();
+
+    BLINK_EXPORT WebString id() const;
+
+    BLINK_EXPORT bool isCircularRegion() const;
+    BLINK_EXPORT WebCircularRegion asCircularRegion() const;

[Peter Beverloo, 2014/07/14 17:23:09]

This would make a copy. Should we instead return a pointer to a casted |this|?

Naming nit: s/as/to/.

[Peter Beverloo, 2014/07/14 18:16:12]

Also needs the Web prefix: toWebCircularRegion, to avoid confusion with the
WebCore CircularRegion type.

+
+#if BLINK_IMPLEMENTATION
+    WebGeofencingRegion(WebCore::GeofencingRegion*);

[Peter Beverloo, 2014/07/14 17:23:09]

This constructor should be marked as explicit.

+    operator WebCore::GeofencingRegion*() const;
+#endif
+
+private:
+    WebPrivatePtr<WebCore::GeofencingRegion> m_private;
+};
+
+} // namespace blink
+
+#endif // WebGeofencingRegion_h