Commit 5b916620 authored by Robert Sesek's avatar Robert Sesek Committed by Chromium LUCI CQ

mac: Reenable message_center_unittests MessagePopupCollectionTest.UpdateContentsCausesPopupClose

This was only failing on 10.10 and Chromium no longer supports that
version of macOS.

Bug: 1047503, 1153883
Change-Id: Icf30557034eb5bcc5bde2a40763901e2d56104a6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2574621
Commit-Queue: Justin DeWitt <dewittj@chromium.org>
Reviewed-by: default avatarJustin DeWitt <dewittj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#834418}
parent d676435f
...@@ -437,14 +437,7 @@ TEST_F(MessagePopupCollectionTest, UpdateContents) { ...@@ -437,14 +437,7 @@ TEST_F(MessagePopupCollectionTest, UpdateContents) {
EXPECT_TRUE(GetPopup(id)->updated()); EXPECT_TRUE(GetPopup(id)->updated());
} }
// Failiing on MacOS 10.10. https://crbug.com/1047503 TEST_F(MessagePopupCollectionTest, UpdateContentsCausesPopupClose) {
#if defined(OS_APPLE)
#define MAYBE_UpdateContentsCausesPopupClose \
DISABLED_UpdateContentsCausesPopupClose
#else
#define MAYBE_UpdateContentsCausesPopupClose UpdateContentsCausesPopupClose
#endif
TEST_F(MessagePopupCollectionTest, MAYBE_UpdateContentsCausesPopupClose) {
std::string id = AddNotification(); std::string id = AddNotification();
AnimateToEnd(); AnimateToEnd();
RunPendingMessages(); RunPendingMessages();
......
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