DevTools: Remove geolocation override from renderer

DevTools: Remove geolocation override from renderer

Geolocation overrides moves from renderer to browser side.

BUG=404995
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183688

[vkuzkokov, 2014-10-03 14:42:00]

vkuzkokov@chromium.org changed reviewers:
+ dgozman@chromium.org

[dgozman, 2014-10-06 11:14:09]

lgtm

[Michael van Ouwerkerk, 2014-10-06 13:08:15]

mvanouwerkerk@chromium.org changed reviewers:
+ mvanouwerkerk@chromium.org

[Michael van Ouwerkerk, 2014-10-06 13:08:16]

lgtm for modules/geolocation - thanks!

[vkuzkokov, 2014-10-06 13:32:27]

vkuzkokov@chromium.org changed reviewers:
+ pfeldman@chromium.org

[vkuzkokov, 2014-10-06 13:32:28]

Pavel, ptal

Source/devtools/protocol.json

[pfeldman, 2014-10-07 08:10:51]

lgtm

[vkuzkokov, 2014-10-07 08:44:59]

Expanded method description as per comment in 603323004.

[vkuzkokov, 2014-10-09 12:45:46]

The CQ bit was checked by vkuzkokov@chromium.org

[commit-bot: I haz the power, 2014-10-09 12:46:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/620343002/20001

[commit-bot: I haz the power, 2014-10-09 13:50:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-09 13:50:53]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)

[vkuzkokov, 2014-10-09 15:57:43]

On 2014/10/09 13:50:53, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   linux_blink_rel on tryserver.blink
>
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)

Seems like geolocation override test should be implemented in browser now.

[dgozman, 2014-10-12 07:41:38]

On 2014/10/09 15:57:43, vkuzkokov wrote:
> On 2014/10/09 13:50:53, I haz the power (commit-bot) wrote:
> > Try jobs failed on following builders:
> >   linux_blink_rel on tryserver.blink
> >
>
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)
> 
> Seems like geolocation override test should be implemented in browser now.

It should not. Content-based browser handlers work in layout test. Please check
what changed here.

[vkuzkokov, 2014-10-13 17:06:27]

Patchset #3 (id:40001) has been deleted

[vkuzkokov, 2014-10-14 13:02:59]

On 2014/10/12 07:41:38, dgozman wrote:
> On 2014/10/09 15:57:43, vkuzkokov wrote:
> > On 2014/10/09 13:50:53, I haz the power (commit-bot) wrote:
> > > Try jobs failed on following builders:
> > >   linux_blink_rel on tryserver.blink
> > >
> >
>
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)
> > 
> > Seems like geolocation override test should be implemented in browser now.
> 
> It should not. Content-based browser handlers work in layout test. Please
check
> what changed here.

I left the tests that didn't require setGeolocationClientMock, altered tests to
to pass validation (-90 <= latitude <= 90 etc.).

[vkuzkokov, 2014-10-14 13:03:08]

The CQ bit was checked by vkuzkokov@chromium.org

[commit-bot: I haz the power, 2014-10-14 13:04:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/620343002/70001

[dgozman, 2014-10-14 13:11:48]

The CQ bit was unchecked by dgozman@chromium.org

[vkuzkokov, 2014-10-14 16:53:17]

The CQ bit was checked by vkuzkokov@chromium.org

[commit-bot: I haz the power, 2014-10-14 16:54:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/620343002/290001

[commit-bot: I haz the power, 2014-10-14 18:15:07]

Committed patchset #5 (id:290001) as 183688