Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(417)

Unified Diff: third_party/WebKit/LayoutTests/geolocation-api/remove-remote-context-in-error-callback-crash.html

Issue 2818563004: sendBeacon should throw a TypeError on bad URLs. (Closed)
Patch Set: Patch Set 2 was the final submit, 3 was uploaded by mistake. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/geolocation-api/remove-remote-context-in-error-callback-crash.html
diff --git a/third_party/WebKit/LayoutTests/geolocation-api/remove-remote-context-in-error-callback-crash.html b/third_party/WebKit/LayoutTests/geolocation-api/remove-remote-context-in-error-callback-crash.html
index 1d27198cc1ee8a67034ae669f2da4586f473a049..b79220fca179bed7a6dd4c6fe5748cfa01ccfd71 100644
--- a/third_party/WebKit/LayoutTests/geolocation-api/remove-remote-context-in-error-callback-crash.html
+++ b/third_party/WebKit/LayoutTests/geolocation-api/remove-remote-context-in-error-callback-crash.html
@@ -4,21 +4,12 @@
<script src="../resources/js-test.js"></script>
<script src="../resources/mojo-helpers.js"></script>
<script src="resources/geolocation-mock.js"></script>
+<script src="../resources/gc.js"></script>
</head>
<body>
<script>
description("Tests that we do not crash when a Geolocation request is made from a remote frame, which is then removed from the DOM in the error callback.");
-function gc() {
- if (window.GCController) {
- GCController.collect();
- return;
- }
-
- for (var i = 0; i < 10000; i++)
- new String(i);
-}
-
function onIframeReady() {
// Make request from remote frame
iframe.contentWindow.navigator.geolocation.getCurrentPosition(function() {

Powered by Google App Engine
This is Rietveld 408576698