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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/security/frameNavigation/not-opener.html

Issue 2042043002: Silencing console messages during some layout tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@replicate-can-open-windows
Patch Set: Added verification of # of opened windows at the end of not-opener test. Created 4 years, 6 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/http/tests/security/frameNavigation/not-opener.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/security/frameNavigation/not-opener.html b/third_party/WebKit/LayoutTests/http/tests/security/frameNavigation/not-opener.html
index bdc9c6984c6a6da4cda2f4a18ee974620ff7b150..ef8277f7e21955b84db2807faa068e78f156f370 100644
--- a/third_party/WebKit/LayoutTests/http/tests/security/frameNavigation/not-opener.html
+++ b/third_party/WebKit/LayoutTests/http/tests/security/frameNavigation/not-opener.html
@@ -3,6 +3,7 @@
<script>
if (window.testRunner) {
testRunner.dumpAsText();
+ testRunner.setDumpConsoleMessages(false);
testRunner.waitUntilDone();
testRunner.setCanOpenWindows();
}
@@ -23,6 +24,7 @@ window.addEventListener("message", function(e) {
} else if (e.data == "pass") {
target.document.body.innerHTML = "Should have navigated this window. It should be still be same-origin.";
log("PASS");
+ log("There are currently " + testRunner.windowCount() + " windows opened.");
target.close();
helper.postMessage("cleanup", "*");
} else if (e.data == "done") {

Powered by Google App Engine
This is Rietveld 408576698