Automatic update from web-platform-testsRevert "Move a test for min-content for a soft-hyphen to wpt"
This reverts commit d37409a17a223249ee661ae504def53bf51b4ce9.
Reason for revert: external/wpt/css/css-sizing/min-content-soft-hyphen-001.html failed on Windows 10
https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Win10/36928
Original change's description:
> Move a test for min-content for a soft-hyphen to wpt
>
> This tests when the inline-size is [min-content]:
> * A `­` produces a break opportunity when computing the
> min-content inline size.
> * The hyphen character generated by `­` is included into
> the min-content inline size.
>
> Originally developed in [CL:1111895].
>
> [min-content]: https://www.w3.org/TR/css-sizing-3/#valdef-width-min-content
> [CL:1111895] https://chromium-review.googlesource.com/c/chromium/src/+/1111895
>
> Bug: 636993
> Change-Id: Ib492b2db023a5169247ca00e5b745e42ec33d197
> Cq-Include-Trybots: luci.chromium.try :linux_layout_tests_layout_ng
> Reviewed-on: https://chromium-review.googlesource.com/1113085
> Commit-Queue: Christian Biesinger <cbiesinger@chromium.org>
> Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#570066}
TBR=cbiesinger@chromium.org,kojii@chromium.org
Change-Id: I3fb91c7c2d66e775a12994734a14afc566d05a3e
No-Presubmit: true
No-Tree-Checks: true
No-Try : true
Bug: 636993
Cq-Include-Trybots: luci.chromium.try :linux_layout_tests_layout_ng
Reviewed-on: https://chromium-review.googlesource.com/1113667
Reviewed-by: Peng Huang <penghuang@chromium.org>
Commit-Queue: Peng Huang <penghuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570109}
--
wpt-commits: ce2196e7a345f209ce095ec6c7356f3a6cf168d4
wpt-pr: 11660
MozReview-Commit-ID: A5fo6hI5Szs
Automatic update from web-platform-testsMove a test for min-content for a soft-hyphen to wpt
This tests when the inline-size is [min-content]:
* A `­` produces a break opportunity when computing the
min-content inline size.
* The hyphen character generated by `­` is included into
the min-content inline size.
Originally developed in [CL:1111895].
[min-content]: https://www.w3.org/TR/css-sizing-3/#valdef-width-min-content
[CL:1111895] https://chromium-review.googlesource.com/c/chromium/src/+/1111895
Bug: 636993
Change-Id: Ib492b2db023a5169247ca00e5b745e42ec33d197
Cq-Include-Trybots: luci.chromium.try :linux_layout_tests_layout_ng
Reviewed-on: https://chromium-review.googlesource.com/1113085
Commit-Queue: Christian Biesinger <cbiesinger@chromium.org>
Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570066}
--
wpt-commits: cb5379322c3ae9f1fb203d6c0b9bc12e7f6af44d
wpt-pr: 11656
MozReview-Commit-ID: 9e7iLpwrmWG
Automatic update from web-platform-testsMerge pull request #11653 from mrego/contain-size-image-alt
[css-contain] Add alt on image at contain-size-013.html
--
wpt-commits: 9d5ca975b959ed3458bd585ff53e705c0551eb62
wpt-pr: 11653
Automatic update from web-platform-testssensors: Yield to the event loop in GenericSensorTestChromium.reset() (#11625)
Do so there instead of in `MockSensorProvider.reset()`, which is now a
regular function.
This brings the code more in line with Blink's own `sensor-helpers.js`, and
fixes one of the issues described in #10906: when a test failed, we'd end up
with the following code path
1. `sensor_tests()`'s calls `GenericSensorTestChromium.reset()`
2. `GenericSensorTestChromium.reset()` calls `MockSensorProvider.reset()`
3. This function `await`s on a promise that resolves inside `setTimeout()`
and yields control back to the event loop
4. The next test is run, and it fails because
`GenericSensorTest.testInternal` hasn't been reset yet.
5. After all tests run, we go back to the `setTimeout()` callbacks and
finish running `GenericSensorTest.reset()`.
which essentially caused all subsequent tests to failure because
`1initialize()` would complain `reset()` had not been run.
We now simplify things, and a test failure results in:
1. `sensor_tests()`'s calls `GenericSensorTestChromium.reset()`
2. `GenericSensorTestChromium.reset()` calls `MockSensorProvider.reset()`,
which runs synchronously and where we close the mojo binding and stop the
mojo interceptor
3. This function `await`s on a promise that resolves inside `setTimeout()`
and yields control back to the event loop to let the mojo shutdown
process finish. The next test may start running before this `sensor_test`
fully finishes, but it's not a problem since we're not running anything
after calling `GenericSensorTestChromium.reset()`
4. The next test runs as expected because `testInternal` has already been
reset
With this change, running `wpt run` with a recent Chromium master build on
accelerometer, generic-sensor, geolocation-sensor, gyroscope, magnetometer
and orientation-sensor yields 249 failures, down from 286.
--
wpt-commits: cc0999e09fac71d8c708445cc9986ee489d12cdb
wpt-pr: 11625
Automatic update from web-platform-testsFetch: split up fetch/api/request/request-keepalive-quota.html
Part of #11571.
--
wpt-commits: f17d9eef1f6b47edc2a6872cff3a53ceceb576b6
wpt-pr: 11593
Automatic update from web-platform-testsES Modules: Use an appropriate referrer policy for module script loading
This CL fixes ReferrerPolicy handling in module script loading. Before this CL,
FetchClientSettingsObjectSnapshot's referrer policy is used, but this is wrong.
The spec requires to use ScriptFetchOptions' referrer policy.
// The "set up the module script request" algorithm:
"Set request's cryptographic nonce metadata to options's cryptographic nonce,
..., and its referrer policy to options's referrer policy."
https://html.spec.whatwg.org/multipage/webappapis.html#set-up-the-module-script-request
Specifically, this CL introduces |referrer_policy_| field in ScriptFetchOptions
class as spec'ed and propagates it to the entire path of module script loading.
ScriptFetchOptions's referrer policy can be set to following values:
- <script>'s "referrerpolicy" attribute is used for <script type='module'>. This
hasn't been implemented yet (see https://crbug.com/841673), so the current
document's referrer policy is used to keep the backward compatibility.
// The "prepare a script" algorithm:
"20. Let referrer policy be the current state of the element's referrerpolicy
content attribute."
"22. Let options be a script fetch options whose cryptographic nonce is
cryptographic nonce, ..., and referrer policy is referrer policy."
https://html.spec.whatwg.org/multipage/scripting.html#prepare-a-script
- <link>'s "referrerpolicy" attribute is used for modulepreload.
// The "Link type "modulepreload"" algorithm:
"9. Let referrer policy be the current state of the element's referrerpolicy
attribute."
"10. Let options be a script fetch options whose cryptographic nonce is
cryptographic nonce, ..., and referrer policy is referrer policy."
https://html.spec.whatwg.org/multipage/links.html#link-type-modulepreload
- The default referrer policy is used for module workers and worklets. This is
the reason why this CL changes test expectations of workers and worklets.
// The "fetch a module worker script graph" algorithm:
"2. Let options be a script fetch options whose cryptographic nonce is the
empty string, ..., and referrer policy is the empty string."
https://html.spec.whatwg.org/multipage/webappapis.html#fetch-a-module-worker-script-tree
Change-Id: Ic0f9e6667cd1b84f74d86fcc948451c1d2f8191f
Bug: 842553, 855963
Reviewed-on: https://chromium-review.googlesource.com/1111743
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569967}
--
wpt-commits: 50b0943d82d36bb92977002bebd5a7990e0ed388
wpt-pr: 11626
Automatic update from web-platform-testswebdriver: remove ES6 constructs from WebDriver tests for IE support (#11637)
IE does not support array destructuring in a `let` statement. This commit
allows the newly added element_click tests to run on IE 11.
--
wpt-commits: f9d0147b58f595f23afb6f66326745e96e105e07
wpt-pr: 11637
Automatic update from web-platform-testsUpdate Web Platform Tests for Feature Policy integration into HTML spec.
Change-Id: Ibfcc3e8bc217aff8782d2aec4c1b73bb4a193e1e
Reviewed-on: https://chromium-review.googlesource.com/1053106
Commit-Queue: Ian Clelland <iclelland@chromium.org>
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569778}
--
wpt-commits: a5facbd6fb610d896eabffbebb45da59edfbff13
wpt-pr: 10966
Automatic update from web-platform-tests[css-grid] Fix positioned items in a grid with calc() gutters
In r543079 we added support for calc() in grid gutters.
In that patch we modified LayoutGrid::GridGap() but we forgot
to do the same in LayoutGrid::AvailableSpaceForGutters().
This patch just changes the IsPercent() call by IsPercentOrCalc()
that way calc() gutters are considered too when looking
for the available space.
BUG=816300
TEST=external/wpt/css/css-grid/abspos/grid-positioned-items-gaps-002.html
TEST=external/wpt/css/css-grid/abspos/grid-positioned-items-gaps-002-rtl.html
Change-Id: I3237d5dc73f508cea6d904b9f1fd3822bb9efe55
Reviewed-on: https://chromium-review.googlesource.com/1112237
Reviewed-by: Sergio Villar <svillar@igalia.com>
Commit-Queue: Manuel Rego Casasnovas <rego@igalia.com>
Cr-Commit-Position: refs/heads/master@{#569733}
--
wpt-commits: 95c2419892175edc877e24672d8bafb8df28cfdd
wpt-pr: 11629
Automatic update from web-platform-tests[css-tables] Make scrollbars visible on mac screenshots
wpt.fyi showed Safari passing these recently-added tests because the
scrollbars don't show in either the tests or their refs.
https://wpt.fyi/css/css-tables/height-distribution/percentage-sizing-of-table-cell-replaced-children-001.html
Aside:
Edge matches Safari's current and Chrome's previous behavior. But
they've confirmed it's a bug:
https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/17505131/
Firefox has a mostly-unrelated incompatibility that doesn't allow us to
check it.
Bug: 837408,838144
Change-Id: I82058d3c22d7668711e9a9a1c977047033d64b46
Reviewed-on: https://chromium-review.googlesource.com/1054770
Commit-Queue: David Grogan <dgrogan@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569698}
--
wpt-commits: 644c41319451e14d31430f659c00bda948ab078e
wpt-pr: 10959
Automatic update from web-platform-testsCorrectly resolve relative URLs for registered custom properties.
Relative URLs in registered custom properties must resolve against the
base URL of the originating stylesheet. For instance, consider a style-
sheet at /style/mystyle.css:
--foo: url("myimage.jpg");
And a document at /index.html:
<style>
background-image: var(--foo);
</style>
If the property --foo is registered with syntax <url>, then the background-
image should be /style/myimage.jpg.
This is contrary to non-registered properties, in which case the
background-image would be /myimage.jpg.
To implement this, this patch scans for URL tokens and 'url('-function
tokens after (var-)resolving the registered custom property
(see CSSVariableResolver::ResolveCustomProperty). The token containing
the relative URL is then rewritten (in-place) to contain an absolute URL
instead.
To avoid doing unnecessary work, we only scan the token stream if
CSSVariableData::needs_variable_resolution_ is set. This is set either
if needs_url_resolution_ is set (because var-references could produce
relative URLs), or if the token stream contains URL/function-tokens.
When a resolved CSSVariableData is created (::CreateResolved), it is
assumed that the incoming token stream contains no URLs that need to be
resolved.
R=futhark@chromium.org, timloh@chromium.org
Bug: 851490
Change-Id: I25b1e839fc92eb538f30670fe91fc92a1ad9d5ea
Reviewed-on: https://chromium-review.googlesource.com/1109975
Commit-Queue: Anders Ruud <andruud@chromium.org>
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569663}
--
wpt-commits: 372c7ad36a7e42a89228aa2bb88b35da5f519fa5
wpt-pr: 11611
Automatic update from web-platform-tests[css-contain] Size containment replaced elements intrinsic size is zero
The spec is very clear about this
(https://drafts.csswg.org/css-contain/#containment-size):
"Replaced elements must be treated as having an intrinsic width
and height of 0."
The patch modifies LayoutReplaced::ComputeIntrinsicSizingInfo()
so that it returns zero for elements with "contain: size".
BUG=853441
TEST=external/wpt/css/css-contain/contain-size-013.html
Change-Id: I91490b120a110be20a91d564da4fdc3d9a11b70a
Reviewed-on: https://chromium-review.googlesource.com/1110819
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Commit-Queue: Manuel Rego Casasnovas <rego@igalia.com>
Cr-Commit-Position: refs/heads/master@{#569610}
--
wpt-commits: 46c8d39a9fde254df3398c681fd724990d289892
wpt-pr: 11622
Automatic update from web-platform-testsCompleted 'unsafe-hashes' per spec
'unsafe-hashed-attributes' renamed to 'unsafe-hashes'
'unsafe-hashes' matches style attributes correctly now
'unsafe-hashes' works for javascript: URLs
'unsafe-hashes' tests added and ammended
spec (approved and to be submitted at the same time as this CR):
https://github.com/w3c/webappsec-csp/pull/311
I2I: https://groups.google.com/a/chromium.org/d/msg/blink-dev/4dohVXDfEI4/tO6rhuv4AwAJ
Bug: 771922
Change-Id: I018cc0f73d492cb4057ff4c41d9be4df8438036c
Reviewed-on: https://chromium-review.googlesource.com/1095217
Reviewed-by: Mike West <mkwst@chromium.org>
Commit-Queue: Andy Paicu <andypaicu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569593}
--
wpt-commits: fb04aed6ac840bd35178406583af6da6759c3566
wpt-pr: 11457
Automatic update from web-platform-testsSec-Metadata: Strings are quoted in structured headers.
So this patch adds quotes.
See https://tools.ietf.org/html/draft-ietf-httpbis-header-structure-06#section-3.7
and 98f7c1253e.
Bug: 843478
Change-Id: I29dfa3a87e3c65a1b64009e173faa6d0f41b40f4
Reviewed-on: https://chromium-review.googlesource.com/1109819
Commit-Queue: Mike West <mkwst@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Reviewed-by: Andy Paicu <andypaicu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569554}
--
wpt-commits: d6020b4e6f97afaa89624f316611061673eb3f53
wpt-pr: 11605
Automatic update from web-platform-tests[testharness.js] Get title from the filename or a META comment if not given (#11403)
--
wpt-commits: f777ec99a11d0d0522e691a8dab93c70d70941f3
wpt-pr: 11403
Automatic update from web-platform-testsWPT: Add long timeout to service-worker/navigation-redirect.https.html test.
This test is slow because it does a lot of navigations.
Bug: 854913
Change-Id: I2b5f984ba4b656af0d735b3c2c14952fdac5867b
Reviewed-on: https://chromium-review.googlesource.com/1111493
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569544}
--
wpt-commits: 91c59efbec2e393f48c9983b5c84acc1aa55efd5
wpt-pr: 11620
Automatic update from web-platform-testsMerge pull request #11619 from frivoal/css-ui-11598
[css-ui] Fix paths to work in testharness & css built test suites
--
wpt-commits: 7d60038cac3d6104be86868a62892e8d1968ddc0
wpt-pr: 11619
--HG--
rename : testing/web-platform/tests/css/css-ui/parsing/resources/parsing-testcommon.js => testing/web-platform/tests/css/css-ui/parsing/support/parsing-testcommon.js
Automatic update from web-platform-testsPrevent abort error from breaking test in FF (fixes#10792)
--
wpt-commits: bbe3411e20381f494983c3acf359bb9fdcaaa16d
wpt-pr: 11614
Automatic update from web-platform-testsMerge pull request #10913 from Tavmjong/paint-context
Add basic tests for context fill and stroke.
--
wpt-commits: e3173905464f0afb36163fa271ff01957eedec7c
wpt-pr: 10913