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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/w3c/webperf/approved/navigation-timing/html/nav2_test_instance_accessors.html

Issue 2675973004: Update NT2 name to be the requested URL instead of 'document' (Closed)
Patch Set: fix failing wpt tests (imported after creation of this CL), and delete a redundant, old test 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
Index: third_party/WebKit/LayoutTests/http/tests/w3c/webperf/approved/navigation-timing/html/nav2_test_instance_accessors.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/w3c/webperf/approved/navigation-timing/html/nav2_test_instance_accessors.html b/third_party/WebKit/LayoutTests/http/tests/w3c/webperf/approved/navigation-timing/html/nav2_test_instance_accessors.html
index cf478b955b1c142bde378a439e3f7a11c90f28eb..77750c2dcbe86476357cea83bccaa6655ba4764e 100644
--- a/third_party/WebKit/LayoutTests/http/tests/w3c/webperf/approved/navigation-timing/html/nav2_test_instance_accessors.html
+++ b/third_party/WebKit/LayoutTests/http/tests/w3c/webperf/approved/navigation-timing/html/nav2_test_instance_accessors.html
@@ -10,7 +10,7 @@
</head>
<body>
<h1>Description</h1>
- <p>This test validates that nav timing 2 instance can be accessed by three different accessors once available: window.performance.getEntries()/getEntriesByType("navigation")/getEntriesByName("document")</p>
+ <p>This test validates that nav timing 2 instance can be accessed by three different accessors once available: window.performance.getEntries()/getEntriesByType("navigation")/getEntriesByName("<Requested URL>")</p>
<script>
var navTiming2Attributes = [
@@ -50,10 +50,10 @@
t.step_func(function (entryList) {
var instance1 = performance.getEntries()[0];
var instance2 = performance.getEntriesByType("navigation")[0];
- var instance3 = performance.getEntriesByName("document")[0];
+ var instance3 = performance.getEntriesByName("http://127.0.0.1:8000/w3c/webperf/approved/navigation-timing/html/nav2_test_instance_accessors.html")[0];
assert_equals(performance.getEntriesByType("navigation").length, 1, "Expected there is only one navigation timing instance.");
- assert_equals(performance.getEntriesByName("document").length, 1, "Expected there is only one navigation timing instance.");
+ assert_equals(performance.getEntriesByName("http://127.0.0.1:8000/w3c/webperf/approved/navigation-timing/html/nav2_test_instance_accessors.html").length, 1, "Expected there is only one navigation timing instance.");
for (var i = 0; i < navTiming2Attributes.length; i++) {
assert_equals(instance1[navTiming2Attributes[i]], instance2[navTiming2Attributes[i]]);

Powered by Google App Engine
This is Rietveld 408576698