[webkit-reviews] review requested: [Bug 41364] [Qt] QWebPage::allowGeolocationRequest should be async API : [Attachment 66042] patch

[bugzilla-daemon at webkit.org]

Mahesh Kulkarni <mahesh.kulkarni at nokia.com> has asked  for review:
Bug 41364: [Qt] QWebPage::allowGeolocationRequest should be async API
https://bugs.webkit.org/show_bug.cgi?id=41364

Attachment 66042: patch
https://bugs.webkit.org/attachment.cgi?id=66042&action=review

------- Additional Comments from Mahesh Kulkarni <mahesh.kulkarni at nokia.com>
Removed : cache client permission from call QWebPage::setUserPermission() even
without any geolocation requests. When Webcore makes a geolocation permission
request, cached permission will be passed on. This way clients can set default
sync permission to geolocation. 

Please review