Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(276)

Unified Diff: Source/modules/geofencing/Geofencing.cpp

Issue 464073002: Pass through geofencing API calls to the content layer. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: fix test expectations Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/modules/geofencing/CircularRegion.idl ('k') | Source/modules/geofencing/GeofencingError.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/geofencing/Geofencing.cpp
diff --git a/Source/modules/geofencing/Geofencing.cpp b/Source/modules/geofencing/Geofencing.cpp
index ad573a525d8ec84752957aa00092b677b530b065..eeb8e6a61b66d6b5d64da0427334703b331c2451 100644
--- a/Source/modules/geofencing/Geofencing.cpp
+++ b/Source/modules/geofencing/Geofencing.cpp
@@ -5,29 +5,88 @@
#include "config.h"
#include "modules/geofencing/Geofencing.h"
+#include "bindings/core/v8/CallbackPromiseAdapter.h"
#include "bindings/core/v8/ScriptPromise.h"
+#include "bindings/core/v8/ScriptPromiseResolver.h"
#include "core/dom/DOMException.h"
#include "core/dom/ExceptionCode.h"
+#include "modules/geofencing/CircularGeofencingRegion.h"
+#include "modules/geofencing/GeofencingError.h"
+#include "modules/geofencing/GeofencingRegion.h"
+#include "public/platform/Platform.h"
+#include "public/platform/WebCircularGeofencingRegion.h"
+#include "public/platform/WebGeofencingProvider.h"
+#include "public/platform/WebGeofencingRegistration.h"
namespace blink {
+namespace {
+
+// For CallbackPromiseAdapter to convert a WebVector of regions to a HeapVector.
+class RegionArray {
+public:
+ typedef blink::WebVector<blink::WebGeofencingRegistration> WebType;
+ static HeapVector<Member<GeofencingRegion> > take(ScriptPromiseResolver* resolver, WebType* regionsRaw)
+ {
+ OwnPtr<WebType> webRegions = adoptPtr(regionsRaw);
+ HeapVector<Member<GeofencingRegion> > regions;
+ for (size_t i = 0; i < webRegions->size(); ++i)
+ regions.append(CircularGeofencingRegion::create((*webRegions)[i].id, (*webRegions)[i].region));
+ return regions;
+ }
+
+ static void dispose(WebType* regionsRaw)
+ {
+ delete regionsRaw;
+ }
+
+private:
+ RegionArray();
+};
+
+} // namespace
+
Geofencing::Geofencing()
{
}
ScriptPromise Geofencing::registerRegion(ScriptState* scriptState, GeofencingRegion* region)
{
- return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError));
+ WebGeofencingProvider* provider = Platform::current()->geofencingProvider();
+ if (!provider)
+ return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError));
+
+ RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);
+ ScriptPromise promise = resolver->promise();
+ // FIXME: somehow pass a reference to the current serviceworker to the provider.
+ provider->registerRegion(region->id(), toCircularGeofencingRegion(region)->webRegion(), new CallbackPromiseAdapter<void, GeofencingError>(resolver));
+ return promise;
}
ScriptPromise Geofencing::unregisterRegion(ScriptState* scriptState, const String& regionId)
{
- return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError));
+ WebGeofencingProvider* provider = Platform::current()->geofencingProvider();
+ if (!provider)
+ return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError));
+
+ RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);
+ ScriptPromise promise = resolver->promise();
+ // FIXME: somehow pass a reference to the current serviceworker to the provider.
+ provider->unregisterRegion(regionId, new CallbackPromiseAdapter<void, GeofencingError>(resolver));
+ return promise;
}
ScriptPromise Geofencing::getRegisteredRegions(ScriptState* scriptState) const
{
- return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError));
+ WebGeofencingProvider* provider = Platform::current()->geofencingProvider();
+ if (!provider)
+ return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError));
+
+ RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);
+ ScriptPromise promise = resolver->promise();
+ // FIXME: somehow pass a reference to the current serviceworker to the provider.
+ provider->getRegisteredRegions(new CallbackPromiseAdapter<RegionArray, GeofencingError>(resolver));
+ return promise;
}
} // namespace blink
« no previous file with comments | « Source/modules/geofencing/CircularRegion.idl ('k') | Source/modules/geofencing/GeofencingError.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698