[webkit-reviews] review requested: [Bug 42629] [QT] Implement client based geolocation for qtport : [Attachment 82587] patch

[bugzilla-daemon at webkit.org]

Mahesh Kulkarni <mahesh.kulkarni at nokia.com> has asked  for review:
Bug 42629: [QT] Implement client based geolocation for qtport
https://bugs.webkit.org/show_bug.cgi?id=42629

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

------- Additional Comments from Mahesh Kulkarni <mahesh.kulkarni at nokia.com>
Updated as per comment 13 and comment 14. 

Follow up bugs raised, 
1. Implementing client-based layout test for qtwebkit -
https://bugs.webkit.org/show_bug.cgi?id=54334
2. Cleanup/Move GeolocationPermissionClientQt.cpp/h implementation to
GeolocationClientQt - https://bugs.webkit.org/show_bug.cgi?id=54338
3. GeolocationPosition to store timestamp as integer in miliseconds -
https://bugs.webkit.org/show_bug.cgi?id=54529