Commit 68a0236a authored by Yoshifumi Inoue's avatar Yoshifumi Inoue Committed by Commit Bot

Revert "Mark BubbleFrameViewTest.RemoveFootnoteView flaky on Mac"

This reverts commit c1ae026a.

Reason for revert: It seems flakiness was caused by bot flakiness.

Original change's description:
> Mark BubbleFrameViewTest.RemoveFootnoteView flaky on Mac
> 
> This patch marks BubbleFrameViewTest.RemoveFootnoteView flaky on Mac
> 
> TBR=mrefaat@chromium.org
> 
> Bug: 903003
> Change-Id: I7028c038018bf1547b92a5c7fb19b5ddf65c3537
> Reviewed-on: https://chromium-review.googlesource.com/c/1325590
> Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
> Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#606328}

TBR=yosin@chromium.org

Change-Id: I39fc11331c1c794763a00bb8a6b82c17dc0f20c5
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 903003
Reviewed-on: https://chromium-review.googlesource.com/c/1325576Reviewed-by: default avatarYoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#606342}
parent cac35f43
......@@ -139,13 +139,7 @@ TEST_F(BubbleFrameViewTest, GetBoundsForClientView) {
EXPECT_EQ(insets.top() + margin_y, frame.GetBoundsForClientView().y());
}
// TODO(crbug.com/903003): We should not mark RemoveFootnoteView flaky.
#if defined(OS_MACOSX)
#define MAYBE_RemoveFootnoteView DISABLED_RemoveFootnoteView
#else
#define MAYBE_RemoveFootnoteView RemoveFootnoteView
#endif
TEST_F(BubbleFrameViewTest, MAYBE_RemoveFootnoteView) {
TEST_F(BubbleFrameViewTest, RemoveFootnoteView) {
TestBubbleFrameView frame(this);
EXPECT_EQ(nullptr, frame.footnote_container_);
View* footnote_dummy_view = new StaticSizedView(gfx::Size(200, 200));
......
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