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

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/clear-site-data/navigation.https.html

Issue 2965173003: Test that Clear-Site-Data is not supported on an insecure navigation (Closed)
Patch Set: Extract populateDatatypes() to TestUtils Created 3 years, 5 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/external/wpt/clear-site-data/navigation.https.html
diff --git a/third_party/WebKit/LayoutTests/external/wpt/clear-site-data/navigation.https.html b/third_party/WebKit/LayoutTests/external/wpt/clear-site-data/navigation.https.html
index e19fc9d3b936aa4eaaa8e76a74eb66308193870e..a8f83903ee00c1415dea975a571533ef017a61ca 100644
--- a/third_party/WebKit/LayoutTests/external/wpt/clear-site-data/navigation.https.html
+++ b/third_party/WebKit/LayoutTests/external/wpt/clear-site-data/navigation.https.html
@@ -8,23 +8,6 @@
<body>
<script>
- /** Ensures that all datatypes are nonempty. */
- function populateDatatypes() {
- return Promise.all(TestUtils.DATATYPES.map(function(datatype) {
- return new Promise(function(resolve, reject) {
- datatype.add().then(function() {
- datatype.isEmpty().then(function(isEmpty) {
- assert_false(
- isEmpty,
- datatype.name +
- " has to be nonempty before the test starts.");
- resolve();
- });
- });
- });
- }));
- }
-
/**
* @param Array.<Array.<Datatype>> combination A combination of datatypes.
* @param Dict.<string, boolean> report A map between a datatype name and
@@ -53,7 +36,7 @@
promise_test(function(test) {
return new Promise(function(resolve_test, reject_test) {
- populateDatatypes()
+ TestUtils.populateDatatypes()
.then(function() {
// Navigate to a resource with a Clear-Site-Data header in
// an iframe, then verify that the correct types have been

Powered by Google App Engine
This is Rietveld 408576698