The stop() method and LayoutTests were added in
https://android-git.corp.google.com/g/#change,53849 which
cherry-picked a WebKit change to attempt to stop leaks. This change
has been rolled back in WebKit pending a better solution, but was
retained in Android as a temporary fix.
Later changes to Geolocation in WebKit have now fixed the underlying
problem and this has been confirmed on Android.
The temporary fix was mostly removed in the merge to WebKit r65615 in
https://android-git.corp.google.com/g/#change,62318 This patch simply
removes the superfluous stop() method and LayoutTests, which were
missed in the above patch.
Bug:
3001031
Change-Id: Id17117f285cb1a8fa600945cc6a87fc336709cab
+++ /dev/null
-Exercises a condition where the DOMWindow may leak if Geolocation requests are ongoing when the page is navigated away or closed. Complete.
+++ /dev/null
-<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
-<html>
-<head>
-<link rel="stylesheet" href="../../js/resources/js-test-style.css">
-<script src="../../js/resources/js-test-pre.js"></script>
-</head>
-<body>
-<p id="description"></p>
-<div id="console"></div>
-<script src="script-tests/ongoing-request-leak.js"></script>
-<script src="../../js/resources/js-test-post.js"></script>
-</body>
-</html>
+++ /dev/null
-description("Exercises a condition where the DOMWindow may leak if Geolocation requests are ongoing when the page is navigated away or closed. The page should reoload.");
-
-if (window.layoutTestController) layoutTestController.setGeolocationPermission(true);
-if (window.layoutTestController) layoutTestController.setMockGeolocationPosition(51.478, -0.166, 100.0);
-
-navigator.geolocation.watchPosition(function() {}, null);
-
-document.body.onload = function() {
- location = "data:text/html,Exercises a condition where the DOMWindow may leak if Geolocation requests are ongoing when the page is navigated away or closed. Complete.<script>if (window.layoutTestController) layoutTestController.notifyDone();</" + "script>";
-}
-
-window.jsTestIsAsync = true;
-window.successfullyParsed = true;
void Geolocation::setIsAllowed(bool) {}
-#if PLATFORM(ANDROID)
-void Geolocation::stop() {}
-#endif // PLATFORM(ANDROID)
-
}
#endif // ENABLE(GEOLOCATION)
void suspend();
void resume();
-#if PLATFORM(ANDROID)
- void stop();
-#endif // PLATFORM(ANDROID)
-
void setIsAllowed(bool);
Frame* frame() const { return m_frame; }