third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_open_blank_page.html - Issue 2647643004: Report nav timing 2 instance as soon as it's requested.

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_open_blank_page.html

Issue 2647643004: Report nav timing 2 instance as soon as it's requested. (Closed)

Patch Set: addressed comments 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

« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_open_data_uri.html » ('j') | third_party/WebKit/Source/core/timing/PerformanceNavigationTiming.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_open_blank_page.html

diff --git a/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_open_blank_page.html b/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_open_blank_page.html

deleted file mode 100644

index dbd5751b94df58a3207a2916ac08b4fdab65b256..0000000000000000000000000000000000000000

--- a/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_open_blank_page.html

+++ /dev/null

@@ -1,28 +0,0 @@

-<!DOCTYPE html>

-<html>

- <head>

- <meta charset="utf-8" />

- <title>Navigation Timing 2 WPT</title>

- <link rel="author" title="Google" href="http://www.google.com/" />

- <link rel="help" href="http://www.w3.org/TR/navigation-timing/#sec-navigation-timing-interface"/>

- <script src="/resources/testharness.js"></script>

- <script src="/resources/testharnessreport.js"></script>

- <script>

- function onload_test()

- {

- var iframe_performance = document.getElementById("frameContext").contentWindow.performance;

- assert_equals(iframe_performance.getEntriesByType("navigation").length, 0, "Expected there is no navigation timing instance");

Kunihiko Sakamoto 2017/02/16 08:42:29 Blank pages and data:url now have navigation timin

sunjian 2017/02/16 22:30:49 I only know that NT1 does expose navigation timing

- done();

- }

- </script>

- </head>

- <body>

- <h1>Description</h1>

- <p>This test validates there does not exist a navigation timing instance when opening internal pages like about:blank.</p>

- <iframe id="frameContext" onload="onload_test();" src="about:blank" style="width: 250px; height: 250px;"></iframe>

- </body>

-</html>