Automatic update from web-platform-tests
[css-flex] Delete broken redundant flex tests
These tests rely on a CJK font such that they pass locally and pass in
chromium's waterfall, but fail on wpt.fyi. The tests rely on the
ordering of glyphs as a pass/fail condition, so using Ahem would make
the tests useless.
But we can just delete these tests because this functionality is tested
by flexbox-writing-mode-0*.html, which we recently imported from
mozilla's tree.
Change-Id: I810196cce08c05545d60744066154b1999426f9f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2718890
Auto-Submit: David Grogan <dgrogan@chromium.org>
Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Commit-Queue: Christian Biesinger <cbiesinger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#858084}
--
wpt-commits: 2d15ebc19f0e1fffc9d7989b784f2ebeb6722410
wpt-pr: 27768
Automatic update from web-platform-tests
Never force legacy layout on NG objects.
Remove dangerous legacy fallback code from LayoutObject(), and do this
safely in the child insertion code instead.
Calling a virtual method in a base class constructor doesn't work.
IsLayoutNGObject() would always return false, since that's how it's
implemented in LayoutObject. This code could cause us to mark an NG
object as requiring legacy fallback, which would make the NG engine
delegate the job to the legacy engine. But the legacy engine would
refuse to do anything, since it's an NG object, so it would delegate to
NG, and so on... Infinite recursion.
The !IsLayoutNGObject() check in LayoutObject() was a mistake added in
CL:2414289, which basically had no effect at all. But we can just remove
the entire thing now, and rely on the new hook in the child insertion
code instead.
Note that the new test doesn't fail without this fix when run normally
by the test runner, because the test needs LayoutNGTextControl to be
disabled in order to fail (so that we attempt legacy fallback), but this
feature is currently set to "test". Unlike the fuzzer, it seems...
Bug: 1181687
Change-Id: I2f174824e4b86d9ae2e5b3b319df7f6036b4f13d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2720104
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#858052}
--
wpt-commits: b71ded018fcecc54f69d7c4e399dd0b9b00eab06
wpt-pr: 27792
Automatic update from web-platform-tests
Add tests for more interesting import assertion module map caching scenarios
Add tests for some scenarios where the module type being part of the
module map cache key (along with the specifier) affects the outcome.
For example, if an import of a given specifier uses the wrong module
type, a later import of the same specifier should succeed if it uses
the correct type assertion.
Add a wpt script for testing some of the stranger cases where repeated
fetches to the same specifier respond with different content types.
Also add basic tests for dynamic import() with the assertions
argument to the virtual/import-assertions suite.
Bug: 1132413
Change-Id: I3ff4cde5a402d3d8c502fdb2bac133323c028254
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2692491
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Domenic Denicola <domenic@chromium.org>
Commit-Queue: Dan Clark <daniec@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#857995}
--
wpt-commits: cb378b9bb94207f789d406c667d5b453aad47920
wpt-pr: 27616
Automatic update from web-platform-tests
[WPT/content-security-policy] Add generated tests
This CL adds generated CSP tests using
/wpt/common/security-features/tools/generate.py
under
/wpt/content-security-policy/gen/,
primarily to add comprehensive test coverage around workers.
This CL only adds tests for
- worker-src and script-src directives
- worker- and script-related subresource requests
- on top-level Documents.
This CL doesn't remove manually-written tests
with similar test coverage, because it would still need
careful comparison of test coverage and
adding more test assertions in
/wpt/content-security-policy/generic/test-case.sub.js.
The failures on Chromium are about the lack of
securitypolicyviolation events due to https://crbug.com/928964.
The failures on other browsers are, as far as I briefly looked at,
about the lack of securitypolicyviolation events, or
due to unimplemented support for underlying features
like (module) SharedWorkers.
Bug: 928964, 1179832
Change-Id: Ib1b81827ab2547a6c59071b2f1a6613c9f9a32e8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2681956
Reviewed-by: Andy Paicu <andypaicu@chromium.org>
Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857969}
--
wpt-commits: bb97a689743eb5e3bc0bc92ac41ab266c54c134e
wpt-pr: 27535
Automatic update from web-platform-tests
Fix NG table collapsed border pixel-snapping and clipping
This fixes the following issues:
- We missed pixel-snapping of the collapsed borders.
Add ObjectPainterBase::DrawBoxSide() which accepts IntRect instead of
float parameters. Previously if the float parameters were not
pixel-snapped, they would be converted to int by truncating which
implicitly snapped to whole pixels incorrectly.
- We missed pixel-snapping and shrinking by border outsets for the clip
when painting background for layered table parts. Combine the clip
logic for cell background and other table part background with
correct pixel-snapping and clip rect, with a test.
Bug: 1180773
Change-Id: I81a967b3de08c116eed97a08e0f608a966088f40
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2717524
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Reviewed-by: Aleks Totic <atotic@chromium.org>
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857955}
--
wpt-commits: e39f98ff5cb658ed5011032a4de05358d4fc3335
wpt-pr: 27765
Automatic update from web-platform-tests
Revert "Repaint when bgcolor animation is triggered"
This reverts commit bbf753e8c7ec7c492f4436c2bdee81278309c4d2.
Reason for revert: Too much flakiness in blink_web_tests on Linux*
Recent failures:
- https://ci.chromium.org/p/chromium/builders/ci/Linux%20Tests/98274
- https://ci.chromium.org/p/chromium/builders/ci/linux-bfcache-rel/15065
- https://ci.chromium.org/p/chromium/builders/ci/linux-trusty-rel/21522
Original change's description:
> Repaint when bgcolor animation is triggered
>
> Right now when a bgcolor animation/transition is triggered, such
> as hovering over an element, we do not request a repaint of the
> element and thus we cannot composite the bgcolor animation.
>
> This CL fixes the problem by triggering a full repaint.
>
> Bug: 1177320
> Change-Id: I87dff5c11d109fc97f41c9b49e13b81613004281
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2698847
> Reviewed-by: Philip Rogers <pdr@chromium.org>
> Reviewed-by: Robert Flack <flackr@chromium.org>
> Commit-Queue: Xida Chen <xidachen@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#857877}
Bug: 1177320
Change-Id: Ifabf8f703af522a6ea25aeac3e76926478a01610
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2719692
Reviewed-by: Jiewei Qian <qjw@chromium.org>
Auto-Submit: Jiewei Qian <qjw@chromium.org>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#857947}
--
wpt-commits: cd176ca0d1fd272b146e586a4051f516f8097b95
wpt-pr: 27793
Automatic update from web-platform-tests
NativeIO: Add quota to sync Blink interface
This CL adds implements NativeIO's quota management system for
synchronous calls in Blink.
Blink requests capacity from the browser process. This CL does not
change how capacity is granted by the browser process. The IPC call to
the browser for requesting capacity is synchronous if and only if
NativeIO's synchronous interface is being used.
The design doc for NativeIO's quota system is here:
https://docs.google.com/document/d/1wUrtCOsyH3qGwKuqLhV9AJD-bDAjAzfPL5r1GT8H4IY
Bug: 1137788
Change-Id: If306e77bdaa66e34c28a633601608de307875c67
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2678951
Commit-Queue: Richard Stotz <rstz@chromium.org>
Auto-Submit: Richard Stotz <rstz@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857916}
--
wpt-commits: 8b759e442fc995202e018cd2d8704a00b5874b62
wpt-pr: 27541
Automatic update from web-platform-tests
Repaint when bgcolor animation is triggered
Right now when a bgcolor animation/transition is triggered, such
as hovering over an element, we do not request a repaint of the
element and thus we cannot composite the bgcolor animation.
This CL fixes the problem by triggering a full repaint.
Bug: 1177320
Change-Id: I87dff5c11d109fc97f41c9b49e13b81613004281
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2698847
Reviewed-by: Philip Rogers <pdr@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857877}
--
wpt-commits: f0ecaebd249dd70c4fe84037e9481eebf2a36ebd
wpt-pr: 27666
Automatic update from web-platform-tests
Split VideoFrameInit into VideoFramePlaneInit. Make optional.
We hadn't been using the VideoFrameInit fields properly and it's
unclear how a user would know these values ahead of time, so just
make the timestamp and duration fields optional while removing all
resolution based fields.
R=chcunningham, sandersd
Bug: 1181894
Test: New tests.
Change-Id: I8ea04d2c1b4e5ef7fadb0a0287cd8adad5715862
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2718896
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Chrome Cunningham <chcunningham@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857829}
--
wpt-commits: 244ff37ec923e10108cfeea90b9304a0b19a86f2
wpt-pr: 27769
Automatic update from web-platform-tests
[BreakoutBox] Add init-based constructor for MSTGenerator
This CL introduces MediaStreamTrackGeneratorInit and a new
constructor that uses it.
As part of the new constructor, the CL:
* Wires implicit signaling.
* Makes the signal buffer size configurable
Bug: 1142955
Change-Id: Ifa816cc60c6b6286a72e26902c5416f5335a6ba6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2696584
Commit-Queue: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: Harald Alvestrand <hta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857762}
--
wpt-commits: 50adeb07eeaec1fef83480a4c922b34f14dc30ac
wpt-pr: 27761
Automatic update from web-platform-tests
Avoid calling AssignedSlot() on pseudo-elements
The repro from [1] causes a scrollevent to be fired on a pseudo-element
inside a shadow tree, which caused a DCHECK at [2]. This CL avoids
that path.
[1] https://bugs.chromium.org/p/chromium/issues/detail?id=1180286#c5
Fixed: 1180286
Change-Id: I28f95d2fa5ad74da9a402a4409f972bab04aec6a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2718566
Auto-Submit: Mason Freed <masonfreed@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Reviewed-by: Joey Arhar <jarhar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857711}
--
wpt-commits: cb2fcc9899df13b277e58cf6d12dc3d627761a4d
wpt-pr: 27767
Automatic update from web-platform-tests
Properly catch exception so that test can run to end (#27752)
Met WebDriverException and it is not properly catched,
and cause an Assertion error in restart_runner, cause
the whole test abort early
Bug: 27751
--
wpt-commits: a177bcf10a9c92e1834954d68641e6e46ae4b863
wpt-pr: 27752
Automatic update from web-platform-tests
Seperate out bidi_session from session in fixtures.py (#27734)
Removed bidi markers and created bidi_session fixtures
Add handling in bidi_session and session start function
to disguish bidi_session and session so that async functions
can be awaited.
Modify tests to have better names.
--
wpt-commits: 8523985b411da9974d4994af1b438a6dafba79c8
wpt-pr: 27734
Automatic update from web-platform-tests
NativeIO: Add quota to async Blink interface
This CL adds a quota management system to NativeIO’s Blink
implementation. Before writing to a file (write, setLength), the
application must explicitly ask for capacity. When deleting or shrinking
files, the freed-up space is given back to the application. Capacity is
allocated per Javascript execution context.
This CL implements the support for the asynchronous methods of NativeIO.
Blink requests capacity from the browser process. Implementing the
browser side will be done in a followup CL. For now, the browser grants
any request up to 8 GiB.
The design doc for NativeIO's quota system is here:
https://docs.google.com/document/d/1wUrtCOsyH3qGwKuqLhV9AJD-bDAjAzfPL5r1GT8H4IY
Bug: 1137788
Change-Id: I9f528373b1e7f5bf45fa8070964cdc3f225d1657
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2675973
Commit-Queue: Richard Stotz <rstz@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857674}
--
wpt-commits: 16e619733e016325afebd30d6cbb97f2caf01d5f
wpt-pr: 27507
Automatic update from web-platform-tests
Split the big test() up into multiple small ones
Also test the computed style on a <picture> with a loaded image.
Addresses the review comments in
https://github.com/web-platform-tests/wpt/pull/27745
--
wpt-commits: 945659ffe92b58c711d68e9c15aa047751dfb89e
wpt-pr: 27745
Automatic update from web-platform-tests
[Private Network Access] Remove addressSpace WPTests.
This CL stops just short of deleting the cors-rfc1918/ top-level
directory, because I believe modifying top-level directories requires a
direct PR to the WPT repository.
Fixed: chromium:1138905
Change-Id: If5db28e60f012b3fa9a11134918bb2bb3a34fede
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2715126
Commit-Queue: Titouan Rigoudy <titouan@chromium.org>
Reviewed-by: Arthur Hemery <ahemery@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857650}
--
wpt-commits: e26ef8aaba60e244454cfa9b5c3cfb0b8b1d027b
wpt-pr: 27743
Automatic update from web-platform-tests
[Credentialless] WPT <iframe>
Add one WPT test about <iframe>.
A few parameters can vary:
- Parent is COEP:{none,credentialless,require-corp}
- Child is COEP:{none,credentialless,require-corp}
- Child is CORP:{undefined, cross-origin}
- Child is {same-origin,cross-origin} with its parent.
Depending on all of those, the iframe must be blocked or must loaded.
Bug: 1175099
Change-Id: I6f60cac9211afdecf9969050efb5e4598cc1052e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2709861
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857616}
--
wpt-commits: c0207bac283ab9bab1711e6cb8999efc8a530afa
wpt-pr: 27719
Automatic update from web-platform-tests
Document multiple wpt_flags
Also, fix the link to file-names.html that was broken.
--
wpt-commits: 05150a5821ff2c22f1f7218f9ec21f6b70c55f6d
wpt-pr: 26844
Automatic update from web-platform-tests
Make streaming upload work with network fallback on service worker
This partially implements https://github.com/whatwg/fetch/pull/1144 and
https://github.com/w3c/ServiceWorker/issues/1560.
ServiceWorkerSubresourceLoader reuses the request body in the original
request in the network fallback case
(ServiceWorkerSubresourceLoader::OnFallback). This is problematic for
a request with streaming upload body, because the body is not copyable.
Ideally we should tee the body as specified in
https://github.com/whatwg/fetch/pull/1144 and
https://github.com/w3c/ServiceWorker/issues/1560, but this CL stops
passing the body to the service worker instead to unblock the origin
trial, assuming that few users care about the streaming body in the
service worker.
Bug: 1165690
Change-Id: Ie1d2d2fd74990b1bf7f9b10e55710a644871cc60
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2712842
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Auto-Submit: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857605}
--
wpt-commits: ecb2781e1fde45e5ed70019f5ac9e14dfec065d1
wpt-pr: 27756
Automatic update from web-platform-tests
NativeIO: Block filenames longer that 100 characters
This fixes a bug where maximum filename length would be
installation-path and host configuration dependent. The proposed limit
is enough for existing partner and test cases. Depending on developer
feedback we might provide a Javascript-based solution as shown in this
document:
https://docs.google.com/document/d/1gjnHg9b0vuTul54MHAzRFfQ6T1XKdayYoGKs6Td8yuQ/edit?ts=5fc7a3ee&pli=1
Bug: 914488
Change-Id: I0b21d2144aee69b6cb82338365a3d6d419e2ca04
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2712793
Commit-Queue: Emanuel Krivoy <krivoy@google.com>
Reviewed-by: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857408}
--
wpt-commits: d0ae73b829a5064cdba3ac4a9f7947dc3e6b73d8
wpt-pr: 27731
Automatic update from web-platform-tests
Simulate actions in click tests in uievents/order-of-events/mouse-events/
Use testdriver Action API to simulate mouse click actions in
third_party/blink/web_tests/external/wpt/uievents/order-of-events/
mouse-events/click-on-body.html,
uievents/order-of-events/mouse-events/click-on-div.html and
uievents/order-of-events/mouse-events/click-on-html.html.
Bug: 1145677
Change-Id: Ief8a399107b044c7f5f8a6c567aa145ded1eaf3e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2713982
Commit-Queue: Lan Wei <lanwei@chromium.org>
Reviewed-by: Mustaq Ahmed <mustaq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857421}
--
wpt-commits: 6ce4e1315d587aa970d37bffa21164a88ffbe0ff
wpt-pr: 27735
Automatic update from web-platform-tests
Fix pointerevents/pointerevent_iframe-touch-action-none_touch.html
We got a script error when running the test
pointerevents/pointerevent_iframe-touch-action-none_touch.html on wpt
dashboard. I found out the problem is "JSON.parse", so I remove it.
https://wpt.fyi/results/pointerevents/
pointerevent_iframe-touch-action-none_touch.html?label=master&
label=experimental&aligned
The reason of the script error is there are some messages from
testdriver-complete which are not a string, they are just objects, so
JSON.parse cannot parse them.
Bug: 1163370
Change-Id: I368a0f0622606cce89b46ee4dc0a0431aef28096
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2702613
Commit-Queue: Lan Wei <lanwei@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857360}
--
wpt-commits: 95924af989a2f3a00cfe3ca173aa83e727fecf63
wpt-pr: 27680
Automatic update from web-platform-tests
NativeIO: Rename top-level symbol to storageFoundation
Rename the top-level symbol from nativeIO to storageFoundation. All
changes were a direct replacement, except for the fixed typo in
third_party/blink/web_tests/external/wpt/native-io/rename_sync_basic.tentative.https.any.js
and the refactored lines that were too long.
Bug: 914488
Change-Id: I30ed7568b9bba0aabde2f79049bb37272cb3bcb0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2716282
Reviewed-by: Mike West <mkwst@chromium.org>
Commit-Queue: Emanuel Krivoy <krivoy@google.com>
Cr-Commit-Position: refs/heads/master@{#857327}
--
wpt-commits: 81254b87afc3b526019317c2ea07dab1bca26623
wpt-pr: 27748
Automatic update from web-platform-tests
[LayoutNG] Paint for "new" columns created by OOFs in nested context
In CL:2704649, support was added for OOF elements that fragment beyond
the last existing fragmentainer in a nested fragmentation context.
Instead of actually creating new columns for these, we add them to the
last existing column with additional inline offset.
In order for paint to work in this case, we needed to ensure that
these fragments retrieve the correct incoming break token in
NGFragmentChildIterator::UpdateSelfFromFragment().
Bug: 1079031
Change-Id: Ib6b7400a5b898431303fe56526d1abced0343a8a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2713259
Commit-Queue: Alison Maher <almaher@microsoft.com>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857315}
--
wpt-commits: 01cc05966ea96ca38b2884e175aed98f6a4f6b7f
wpt-pr: 27749
Automatic update from web-platform-tests
Simulate actions in mouse tests in uievents/order-of-events/mouse-events/
Use testdriver Action API to simulate mouse click actions in
third_party/blink/web_tests/external/wpt/uievents/order-of-events/
mouse-events/mouseevents-mousemove.htm,
uievents/order-of-events/mouse-events/mousemove-across.html and
uievents/order-of-events/mouse-events/mousemove-between.html.
Bug: 1145677
Change-Id: I8a9bb41496651b6751fe88f84b693380b2d9531d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2713430
Reviewed-by: Mustaq Ahmed <mustaq@chromium.org>
Commit-Queue: Lan Wei <lanwei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#857268}
--
wpt-commits: ac7d1c204a052dcf97616139ea6f2b0804799164
wpt-pr: 27738