We got the data we need to make a decision.
(( This is a combined revert of the four commits that added the instrumentation ))
Differential Revision: https://phabricator.services.mozilla.com/D94620
We want to partition the CORS preflight cache by the network state
partitioning. So, we have done two things in the patch. First, we make
the CORS preflight cache to be aware of the OriginAttributes. Second, we
use the originAttributes of the network state partitioning in the CORS
preflight cache. As the result, the CORS preflight cache will be
partitioned by the originAttributes.partitionKey as well as other
originAttributes fields if present.
Differential Revision: https://phabricator.services.mozilla.com/D93683
Private class methods define an internal lambda function for the initializer,
but it currently uses source coordinates of [0,0) which isn't very helpful.
This patch updates it to cover the full field name until end of method body.
These distinctions are generally not exposed to users, but having internal
consistency is still helpful for defining engine invariants.
Differential Revision: https://phabricator.services.mozilla.com/D94612
Add tests to document the BaseScripts generated for different parse
constructs and track the sourceStart/sourceEnd values. This uses the debugger
API to find hidden scripts and access the source values. The details this
test checks are arbitrary implementation details, and the test expectations
can be updated as necessary.
Differential Revision: https://phabricator.services.mozilla.com/D94611
The method is tested by `test_nsITableEditor_getFirstSelectedCellInTable.html`.
And now, nobody uses `CellAndIndexes` so that this patch removes it.
Depends on D94239
Differential Revision: https://phabricator.services.mozilla.com/D94240
The default fonts of CJK on Samsung's Galaxy device seems to be SEC CJK fonts.
This seems to be modified version of Noto Sans CJK. Samsung has No. 1 market
share for Android, so we should add this fonts to default font list.
Differential Revision: https://phabricator.services.mozilla.com/D94702
This patch modifies the cron to run chrome linux tests on browsertime instead of webext. It also removes the linux smoke tests.
Differential Revision: https://phabricator.services.mozilla.com/D94628
Automatic update from web-platform-tests
webrtc: add WPT test for handling of unknown media types
asserting that a media type which is not understood is still kept in the
answer.
Bug: webrtc:3513
Change-Id: Ice96c09d7d1990dba677095fe882aab0fbe1a91d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2270180
Reviewed-by: Harald Alvestrand <hta@chromium.org>
Commit-Queue: Philipp Hancke <philipp.hancke@googlemail.com>
Cr-Commit-Position: refs/heads/master@{#820298}
--
wpt-commits: 304915c05721c2dbbf0f439618e1125b343f258a
wpt-pr: 24357
Automatic update from web-platform-tests
Remove invalid relative import in vcs.py (#26262)
It was introduced by e98eee5, but absolute import alone works on both Py2/3.
Closes#26071
--
wpt-commits: 3db9eb442bedcc796ab7ecd1666cc1b6682e103a
wpt-pr: 26262
Automatic update from web-platform-tests
Revert "[mathml] Implement special drawing of largeop operator in displaystyle"
This reverts commit 830b5b3af88cbe3fc7e6169c9772fdebe01f6797.
Reason for revert: Causing crashes in several mathml tests on the MSAN bot. First failing build: https://ci.chromium.org/p/chromium/builders/ci/WebKit%20Linux%20MSAN/8001
Original change's description:
> [mathml] Implement special drawing of largeop operator in displaystyle
>
> This implements special drawing of largeop <mo> element in display style
> as described in step 3. of [1]. A new algorithm is introduced for that
> purpose and in particular only used for <mo> elements that are made of
> a single glyph. Existing painting logic for stretchy radical symbol is
> reused and adapted to work with largeop operators.
>
> [1] https://mathml-refresh.github.io/mathml-core/#layout-of-operators
>
> Bug: 6606
> Change-Id: Icca70f4bf464417809970fecc101e5aa7f149621
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2414298
> Commit-Queue: Rob Buis <rbuis@igalia.com>
> Reviewed-by: Koji Ishii <kojii@chromium.org>
> Reviewed-by: Frédéric Wang <fwang@igalia.com>
> Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#819855}
TBR=wangxianzhu@chromium.org,kojii@chromium.org,ikilpatrick@google.com,ikilpatrick@chromium.org,fwang@igalia.com,mstensho@chromium.org,rbuis@igalia.com
Change-Id: I97a8a0708d5acd3d43fd67a2deff3fed5967fc5e
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 6606
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2495061
Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Commit-Queue: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#820204}
--
wpt-commits: 0c6b6ed8b55e50fbf5669874815bd6ea0eb1c5a9
wpt-pr: 26254
Automatic update from web-platform-tests
Fix NGLineBreaker::RewindOverflow crash
This patch is a re-fix of r819722 <crrev.com/c/2489730>.
It turned out that |NGLineBreaker| hangs the attached test
case before r819722, and crashes after. This is because
|Rewind| may fail if the line has floats, and in that case,
the current item is not a text item.
This patch changes it to just setting the state to |kTrailing|
and return, so that, if |Rewind| succeeds, |BreakLine| will
call |HandleText|, which will call |HandleTrailingSpaces| when
the text item starts with spaces.
When |Rewind| fails, |BreakLine| can handle non-text items too.
Bug: 1141384
Change-Id: I96e428d2c9ff50b3685c2a83eafdcea509749594
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2492700
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#820154}
--
wpt-commits: 382094f140d39f6fa8b14ab15107bed84317e89a
wpt-pr: 26248
Automatic update from web-platform-tests
[beacon] Fix ArrayBuffer and URLSearchParams data
`navigator.sendBeacon()` was not handling DOMArrayBuffer and
URLSearchParams inputs properly, resulting in failing WPTs related to
the sent Content-Type, as well as in the wrong data sent in the case
of DOMArrayBuffers.
This CL fixes that.
Bug: 876671
Change-Id: I17674b3041aa0f0bdbd1a570ab34be48b0dd98b4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2489986
Commit-Queue: Yoav Weiss <yoavweiss@chromium.org>
Reviewed-by: Adam Rice <ricea@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#820134}
--
wpt-commits: 4a65f5002f1e2e778f62a9b99f4298dc8948f26c
wpt-pr: 26214
Automatic update from web-platform-tests
2d canvas: drawImage() does not throw "IndexSizeError" when source width/height is zero
Fixes#2835.
--
wpt-commits: c71e3bf23f8a328f68614fc1716d2659f2547eda
wpt-pr: 26246
Automatic update from web-platform-tests
HTMLPluginElement: Add object to frame view's update part if locked.
This patch ensures that when we delay the load because the plugins
need to be loaded and we're under a content-visibility lock, then
we add the object to be updated into frame view's list immediately
This is typically done during layout, but if layout is prevented then
we run into a possibility of delaying load event forever. Adding
the object to the update set immediately avoids the problem.
R=masonfreed@chromium.org
Bug: 1138128
Change-Id: I4d918c6bbdcd80aa6f01bf3b068acb64a93b0196
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2485671
Commit-Queue: vmpstr <vmpstr@chromium.org>
Reviewed-by: Mason Freed <masonfreed@chromium.org>
Reviewed-by: Stefan Zager <szager@chromium.org>
Cr-Commit-Position: refs/heads/master@{#820068}
--
wpt-commits: 3dd0797f48bc867df582d7e6d5673622e3593ed3
wpt-pr: 26193
Automatic update from web-platform-tests
Remove sorting and de-duplication in media queries.
This was removed from the spec per resolution[1] and incompatible with
never media queries. Improves interop with Gecko which have not seen any
issues with the different serialization.
Removed fast/media test which is covered by existing wpt tests.
[1] https://github.com/w3c/csswg-drafts/issues/5627#issuecomment-712475204
Bug: 1138859
Change-Id: I1483008c81df90f8277dcad7e90c8036c5cc019b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2478992
Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#819090}
--
wpt-commits: d004bddde373550fe9a563487adda3e17fc4c697
wpt-pr: 26242