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

Unified Diff: third_party/WebKit/LayoutTests/fast/loader/crash-focus-in-unload.html

Issue 2692423006: Fix stale focusedElement in unloaded document. (Closed)
Patch Set: Check nullptr. Created 3 years, 10 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/dom/Element.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/fast/loader/crash-focus-in-unload.html
diff --git a/third_party/WebKit/LayoutTests/fast/loader/crash-focus-in-unload.html b/third_party/WebKit/LayoutTests/fast/loader/crash-focus-in-unload.html
new file mode 100644
index 0000000000000000000000000000000000000000..8510868571a9352cc1c7d3eec663abc734d1ce0e
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/loader/crash-focus-in-unload.html
@@ -0,0 +1,30 @@
+<!DOCTYPE html>
+<script src='../../resources/testharness.js'></script>
+<script src='../../resources/testharnessreport.js'></script>
+<body>
+<iframe></iframe>
+</body>
+<script>
+'use strict';
+
+let iframe = document.querySelector('iframe');
+let iframeDoc = iframe.contentDocument;
+
+let focusableIframe = document.createElement('iframe');
+iframeDoc.body.appendChild(focusableIframe);
+
+focusableIframe.contentWindow.addEventListener('unload', () => {
+ // Try to focus on frame owner during unload event of its content.
+ focusableIframe.focus();
+});
+
+test(() => {
+ // Trigger unload event.
+ document.body.appendChild(focusableIframe);
+
+ var input = document.createElement('input');
+ iframeDoc.body.appendChild(input);
+ input.focus();
+ assert_true(true, 'This test must run without crash.');
+}, 'Should not trigger assertion. crbug.com/677690');
+</script>
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/dom/Element.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698