Skip to content

Commit

Permalink
Set Origin-Agent-Cluster header for 3 tests.
Browse files Browse the repository at this point in the history
The WebPagePopupImpl constructor has special handling for origin agent
clusters to support testing, that these particular tests rely on. This
mechanism will break if Origin-Agent-Clustersare enabled by default.
Hence we opt-out of OAC by setting the appropriate header on these
tests (and their support resources). This will keep the tests functional.

Bug: 1259920
Change-Id: I4ae3a90acdad57a5b7ec632bfd674c8be966d61f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4110839
Reviewed-by: Nate Chapin <japhet@chromium.org>
Commit-Queue: Daniel Vogelheim <vogelheim@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1084724}
  • Loading branch information
otherdaniel authored and Chromium LUCI CQ committed Dec 17, 2022
1 parent 91db9fa commit ad4c21c
Show file tree
Hide file tree
Showing 6 changed files with 18 additions and 3 deletions.
2 changes: 1 addition & 1 deletion third_party/blink/web_tests/SlowTests
Expand Up @@ -564,7 +564,7 @@ crbug.com/1093849 external/wpt/dom/nodes/Element-classlist.html [ Slow ]
crbug.com/1093853 external/wpt/dom/ranges/Range-surroundContents.html [ Slow ]
crbug.com/1093478 external/wpt/quirks/unitless-length/limited-quirks.html [ Slow ]
crbug.com/1133836 external/wpt/scroll-to-text-fragment/redirects.html [ Slow ]
crbug.com/1134580 http/tests/eye-dropper/color-picker-show-eye-dropper.html [ Slow ]
crbug.com/1134580 http/tests/eye-dropper/color-picker-show-eye-dropper.php [ Slow ]
crbug.com/1145716 fast/forms/calendar-picker/datetimelocal-picker-open-to-focused-field.html [ Slow ]

# These began to hit timeouts when moving from Windows-10-15063 to Windows-10-18363
Expand Down
@@ -1,3 +1,6 @@
<?php
header("Origin-Agent-Cluster: ?0");
?>
<!DOCTYPE html>
<html>
<head>
Expand Down
@@ -1,3 +1,6 @@
<?php
header("Origin-Agent-Cluster: ?0");
?>
<!DOCTYPE html>
<html>
<head>
Expand All @@ -9,7 +12,7 @@
let t = async_test("Test input color popup's keyboard usability when inside cross-process iframe.");

let iframe = document.createElement("iframe");
iframe.src = "http://localhost:8000/forms/resources/color-picker-keyboard-cross-domain-iframe.html";
iframe.src = "http://localhost:8000/forms/resources/color-picker-keyboard-cross-domain-iframe.php";

const runTest = t.step_func((event) => {
// The eventSender.keyDown() invocations in the iframe create extra window messages.
Expand Down
@@ -1,3 +1,6 @@
<?php
header("Origin-Agent-Cluster: ?0");
?>
<!DOCTYPE html>
<html>
<head>
Expand All @@ -9,7 +12,7 @@
let t = async_test("Test input date popup's keyboard usability when inside cross-process iframe.");

let iframe = document.createElement("iframe");
iframe.src = "http://localhost:8000/forms/resources/date-picker-keyboard-cross-domain-iframe.html";
iframe.src = "http://localhost:8000/forms/resources/date-picker-keyboard-cross-domain-iframe.php";

const runTest = t.step_func((event) => {
// The eventSender.keyDown() invocations in the iframe create extra window messages.
Expand Down
@@ -1,3 +1,6 @@
<?php
header("Origin-Agent-Cluster: ?0");
?>
<html>
<input type="color" id="color" value="#126465">

Expand Down
@@ -1,3 +1,6 @@
<?php
header("Origin-Agent-Cluster: ?0");
?>
<html>
<input type="date" id="date" value="2019-12-12">

Expand Down

0 comments on commit ad4c21c

Please sign in to comment.