Have WebGeolocationController always wrap up its private controller object

Have WebGeolocationController always wrap up its private controller object

Until Oilpan is firmly enabled, always use GeolocationControllerPrivate
to wrap up the non-owned GeolocationController* object. The previous
implementation was performing an untidy downcast with undefined behavior;
no need for such shortcuts here.

R=
BUG=581112
Committed: https://crrev.com/209cd7deba6500a0dc82ffbd48bc9522cdf9c1b2
Cr-Commit-Position: refs/heads/master@{#371341}

[sof, 2016-01-25 20:41:33]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-01-25 20:42:09]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1638573002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1638573002/1

[sof, 2016-01-25 20:43:45]

sigbjornf@opera.com changed reviewers:
+ oilpan-reviews@chromium.org

[sof, 2016-01-25 20:43:45]

please take a look.

This GeolocationControllerPrivate intermediary will be shortlived iff Oilpan
sticks, but best make it correct while it is still needed.

[commit-bot: I haz the power, 2016-01-25 22:00:03]

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

[commit-bot: I haz the power, 2016-01-25 22:00:04]

Dry run: This issue passed the CQ dry run.

[haraken, 2016-01-25 22:44:42]

LGTM

[haraken, 2016-01-25 22:44:48]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-01-25 22:47:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1638573002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1638573002/1

[commit-bot: I haz the power, 2016-01-25 22:57:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-25 22:59:27]

Description was changed from

==========
Have WebGeolocationController always wrap up its private controller object

Until Oilpan is firmly enabled, always use GeolocationControllerPrivate
to wrap up the non-owned GeolocationController* object. The previous
implementation was performing an untidy downcast with undefined behavior;
no need for such shortcuts here.

R=
BUG=581112
==========

to

==========
Have WebGeolocationController always wrap up its private controller object

Until Oilpan is firmly enabled, always use GeolocationControllerPrivate
to wrap up the non-owned GeolocationController* object. The previous
implementation was performing an untidy downcast with undefined behavior;
no need for such shortcuts here.

R=
BUG=581112

Committed: https://crrev.com/209cd7deba6500a0dc82ffbd48bc9522cdf9c1b2
Cr-Commit-Position: refs/heads/master@{#371341}
==========

[commit-bot: I haz the power, 2016-01-25 22:59:28]

Patchset 1 (id:??) landed as
https://crrev.com/209cd7deba6500a0dc82ffbd48bc9522cdf9c1b2
Cr-Commit-Position: refs/heads/master@{#371341}