Chromium Code Reviews Chromium Code Reviews
Issue 2774543003:
Fix PerformanceNavigationTiming accessor behavior after document detach. (Closed)
| Index: third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_document_replaced.html |
| diff --git a/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_document_replaced.html b/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_document_replaced.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..6e81d4bc74423368c2acfbd47a3dff53d9b551c9 |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_document_replaced.html |
| @@ -0,0 +1,61 @@ |
| +<!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> |
| + var navigation_frame; |
| + var pnt1; |
| + var step = 1; |
| + function onload_test() |
| + { |
| + navigation_frame = document.getElementById("frameContext").contentWindow; |
| + if (step === 1) { |
| + pnt1 = navigation_frame.performance.getEntriesByType("navigation")[0]; |
|
Nate Chapin
2017/03/27 21:44:35
Nit: move this to the case 1 in the switch stateme
sunjian
2017/03/27 23:07:05
Done.
|
| + } |
| + setTimeout("nav_frame();", 100); |
|
Nate Chapin
2017/03/27 21:44:35
Is a 100ms setTimeout() necessary? At least in chr
sunjian
2017/03/27 23:07:05
Acknowledged.
|
| + } |
| + |
| + function nav_frame() |
| + { |
| + switch (step) |
| + { |
| + case 1: |
| + { |
| + navigation_frame.location.href = '/navigation-timing/resources/blank_page_green_with_onunload.html'; |
| + step++; |
| + break; |
| + } |
| + case 2: |
| + { |
| + navigation_frame.history.back(); |
| + step++; |
| + break; |
| + } |
| + case 3: |
| + { |
| + var pnt2 = navigation_frame.performance.getEntriesByType("navigation")[0]; |
| + assert_equals(pnt1.type, "navigate"); |
| + assert_equals(pnt2.type, "back_forward"); |
| + done(); |
| + break; |
| + } |
| + default: |
| + break; |
| + } |
| + } |
| + </script> |
| + </head> |
| + <body> |
| + <h1> |
| + Description</h1> |
| + <p> |
| + This test validates two different PerformanceNavigationTiming instances don't reference the same DocumentLoader when computing their |
|
Nate Chapin
2017/03/27 21:44:35
This test will get upstreamed to wpt, right? You p
sunjian
2017/03/27 23:07:05
Acknowledged.
|
| + type field </p> |
| + <iframe id="frameContext" onload="onload_test();" src="resources/blank_page_yellow_with_onunload.html" style="width: 250px; height: 250px;"></iframe> |
| + </body> |
| +</html> |
