DescriptionMerge 66886 - 2010-09-07 Jonathan Dixon <joth@chromium.org>
Reviewed by Jeremy Orlow.
Access to out-of-scope WebGeolocationServiceBridgeImpl
https://bugs.webkit.org/show_bug.cgi?id=45112
Add missing virtual destructor to the abstract base class.
* platform/chromium/GeolocationServiceChromium.cpp:
(WebCore::GeolocationServiceBridge::~GeolocationServiceBridge):
* platform/chromium/GeolocationServiceChromium.h:
2010-09-07 Jonathan Dixon <joth@chromium.org>
Reviewed by Jeremy Orlow.
Access to out-of-scope WebGeolocationServiceBridgeImpl
https://bugs.webkit.org/show_bug.cgi?id=45112
Keep the WebGeolocationService pointer for lifetime of the bridge, to
ensure it can be detached reliably
* public/WebGeolocationServiceBridge.h:
(WebKit::WebGeolocationServiceBridge::~WebGeolocationServiceBridge):
* src/WebGeolocationServiceBridgeImpl.cpp:
(WebKit::WebGeolocationServiceBridgeImpl::WebGeolocationServiceBridgeImpl):
(WebKit::WebGeolocationServiceBridgeImpl::~WebGeolocationServiceBridgeImpl):
(WebKit::WebGeolocationServiceBridgeImpl::startUpdating):
(WebKit::WebGeolocationServiceBridgeImpl::stopUpdating):
(WebKit::WebGeolocationServiceBridgeImpl::suspend):
(WebKit::WebGeolocationServiceBridgeImpl::resume):
(WebKit::WebGeolocationServiceBridgeImpl::attachBridgeIfNeeded):
BUG=53394
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=67097
Patch Set 1 #
Messages
Total messages: 1 (0 generated)
|