Commit 2daa3730 authored by Koji Ishii's avatar Koji Ishii Committed by Commit Bot

Revert "Disable two failing tests in NGInlineCursorTest"

This reverts commit f2520c7f.

Reason for revert: The failures are only on "refs/branch-heads/3945",
not on master branch. See crbug.com/1039506#c11

Original change's description:
> Disable two failing tests in NGInlineCursorTest
> 
> Failing on android-arm64-stable-tests:
> NGInlineCursorTest/NGInlineCursorTest.CulledInlineWithAtomicInline/1
> NGInlineCursorTest/NGInlineCursorTest.CulledInlineWithFloat/1
> 
> BUG=1046338
> NOTRY=true
> TBR=chouinard@chromium.org,kojii@chromium.org
> 
> Change-Id: Ic4c94abe09498302c813bb4d48a47a28cc6a3bd4
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2023494
> Commit-Queue: Tim Volodine <timvolodine@chromium.org>
> Reviewed-by: Tim Volodine <timvolodine@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#735851}

TBR=timvolodine@chromium.org,kojii@chromium.org,chouinard@chromium.org

Change-Id: Icf33bdc9750c93701206666e346110fe7ca228e7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1046338
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2025748Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Reviewed-by: default avatarTim Volodine <timvolodine@chromium.org>
Commit-Queue: Tim Volodine <timvolodine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#736456}
parent e41aab5f
...@@ -204,8 +204,7 @@ TEST_P(NGInlineCursorTest, ContainingLine) { ...@@ -204,8 +204,7 @@ TEST_P(NGInlineCursorTest, ContainingLine) {
EXPECT_EQ(line2, cursor); EXPECT_EQ(line2, cursor);
} }
// Failing on a buildbot, http://crbug.com/1046338 TEST_P(NGInlineCursorTest, CulledInlineWithAtomicInline) {
TEST_P(NGInlineCursorTest, DISABLED_CulledInlineWithAtomicInline) {
SetBodyInnerHTML( SetBodyInnerHTML(
"<div id=root>" "<div id=root>"
"<b id=culled>abc<div style=display:inline>ABC<br>XYZ</div>xyz</b>" "<b id=culled>abc<div style=display:inline>ABC<br>XYZ</div>xyz</b>"
...@@ -226,8 +225,7 @@ TEST_P(NGInlineCursorTest, DISABLED_CulledInlineWithAtomicInline) { ...@@ -226,8 +225,7 @@ TEST_P(NGInlineCursorTest, DISABLED_CulledInlineWithAtomicInline) {
// We should not have float:right fragment, because it isn't in-flow in // We should not have float:right fragment, because it isn't in-flow in
// an inline formatting context. // an inline formatting context.
// For https://crbug.com/1026022 // For https://crbug.com/1026022
// Failing on a buildbot, http://crbug.com/1046338 TEST_P(NGInlineCursorTest, CulledInlineWithFloat) {
TEST_P(NGInlineCursorTest, DISABLED_CulledInlineWithFloat) {
SetBodyInnerHTML( SetBodyInnerHTML(
"<div id=root>" "<div id=root>"
"<b id=culled>abc<div style=float:right></div>xyz</b>" "<b id=culled>abc<div style=float:right></div>xyz</b>"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment