According to https://github.com/crosswalk-project/chromium-crosswalk/blob/708fc322d1336a59ca2f21c33da5649db1e1ee99/content/browser/geolocation/geolocation_dispatcher_host.cc#L147, if we return NULL here, `GeolocationDispatcherHostImpl::OnRequestPermission` will allow all permission requests by returning `true`. It is the current behavior. As you mentioned, in this implementation we also grant all permission requests. If my understanding is correct, this implementation is to bridge the permission handling to Java side. So my question is what's the root cause of https://crosswalk-project.org/jira/browse/XWALK-151? How does this patch fix that? Thanks.