From e07e95f1c09d026b6c55b51cbfb438eec8d8a21b Mon Sep 17 00:00:00 2001 From: Bret Sepulveda Date: Sat, 24 Oct 2020 10:43:53 +0000 Subject: [PATCH] Bug 1672969 [wpt PR 26254] - Revert "[mathml] Implement special drawing of largeop operator in displaystyle", a=testonly MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 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 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 > Reviewed-by: Koji Ishii > Reviewed-by: Frédéric Wang > Reviewed-by: Ian Kilpatrick > 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 Commit-Queue: Bret Sepulveda Cr-Commit-Position: refs/heads/master@{#820204} -- wpt-commits: 0c6b6ed8b55e50fbf5669874815bd6ea0eb1c5a9 wpt-pr: 26254 --- .../operators/largeop-hit-testing.html | 63 ------------------- .../padding-border-margin-002-ref.html | 18 ------ .../padding-border-margin-002.html | 34 ---------- 3 files changed, 115 deletions(-) delete mode 100644 testing/web-platform/tests/mathml/presentation-markup/operators/largeop-hit-testing.html delete mode 100644 testing/web-platform/tests/mathml/relations/css-styling/padding-border-margin/padding-border-margin-002-ref.html delete mode 100644 testing/web-platform/tests/mathml/relations/css-styling/padding-border-margin/padding-border-margin-002.html diff --git a/testing/web-platform/tests/mathml/presentation-markup/operators/largeop-hit-testing.html b/testing/web-platform/tests/mathml/presentation-markup/operators/largeop-hit-testing.html deleted file mode 100644 index 4f6a7782f279..000000000000 --- a/testing/web-platform/tests/mathml/presentation-markup/operators/largeop-hit-testing.html +++ /dev/null @@ -1,63 +0,0 @@ - - - - -Test hit testing on large operators - - - - - - - - - - - - - - ⫿ - - - - - ⫿ - - - - - - - diff --git a/testing/web-platform/tests/mathml/relations/css-styling/padding-border-margin/padding-border-margin-002-ref.html b/testing/web-platform/tests/mathml/relations/css-styling/padding-border-margin/padding-border-margin-002-ref.html deleted file mode 100644 index e13a9f47ff4f..000000000000 --- a/testing/web-platform/tests/mathml/relations/css-styling/padding-border-margin/padding-border-margin-002-ref.html +++ /dev/null @@ -1,18 +0,0 @@ - - - - -Padding/border/margin on largeop (reference) - -

This test passes if you see a cyan rectangle of width 300px and - height 1500px, surrounded by a 10px blue padding, surrounded by a 10px - blue/yellow dashed border, itself - surrounded by a 10px pink margin.

- -
-
-
-
-
- - diff --git a/testing/web-platform/tests/mathml/relations/css-styling/padding-border-margin/padding-border-margin-002.html b/testing/web-platform/tests/mathml/relations/css-styling/padding-border-margin/padding-border-margin-002.html deleted file mode 100644 index 1ccfbb7005fa..000000000000 --- a/testing/web-platform/tests/mathml/relations/css-styling/padding-border-margin/padding-border-margin-002.html +++ /dev/null @@ -1,34 +0,0 @@ - - - - -Padding/border/margin on largeop - - - - - - - -

This test passes if you see a cyan rectangle of width 300px and - height 1500px, surrounded by a 10px blue padding, surrounded by a 10px - blue/yellow dashed border, itself - surrounded by a 10px pink margin.

-
- - ⫿ - -
- -