From 930792dfdde9ad0fac3251ab70eb00d97f2fe88d Mon Sep 17 00:00:00 2001 From: Steve Block Date: Thu, 18 Feb 2010 11:26:48 +0000 Subject: Fixes a bug with the Geolocation suspend/resume behaviour PlatformBridge::isWebViewPaused needs to be an instance method, rather than a static. This fixes a bug where if the user switches browser windows while a page that uses Geolocation is still loading, the Geolocation service won't be started in the suspended state. Note that this is a temporary fix, as the upstreaming of the existing suspend/resume code will introduce a new approach, which will avoid this problem altogether. See https://android-git.corp.google.com/g/#change,38942 Change-Id: I3f07f8837b8a8c1c5e7e4f5112ab487188670c3a --- WebKit/android/WebCoreSupport/PlatformBridge.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'WebKit/android/WebCoreSupport') diff --git a/WebKit/android/WebCoreSupport/PlatformBridge.cpp b/WebKit/android/WebCoreSupport/PlatformBridge.cpp index b91a5d8..c04600d 100644 --- a/WebKit/android/WebCoreSupport/PlatformBridge.cpp +++ b/WebKit/android/WebCoreSupport/PlatformBridge.cpp @@ -113,9 +113,10 @@ NPObject* PlatformBridge::pluginScriptableObject(Widget* widget) #endif } -bool PlatformBridge::isWebViewPaused() +bool PlatformBridge::isWebViewPaused(const WebCore::FrameView* frameView) { - return WebViewCore::isPaused(); + android::WebViewCore* webViewCore = android::WebViewCore::getWebViewCore(frameView); + return webViewCore->isPaused(); } bool PlatformBridge::popupsAllowed(NPP) -- cgit v1.1