DescriptionRevert of Port NotificationViewMD unit tests from NotificationViewTest. (patchset #5 id:80001 of https://codereview.chromium.org/2966343002/ )
Reason for revert:
message_center_unittests failing on chromium.memory/Linux Chromium OS ASan LSan Tests (1)
BUG=739356
Original issue's description:
> Port NotificationViewMD unit tests from NotificationViewTest.
>
> Ported unit tests from NotificationViewTest and created
> NotificationViewMDTest. Also fixed bugs and check failures found during
> porting the tests.
> * Fix RemoveChildView null checking. (CHECK fails on debug builds.)
> * Fix action button hover state inheritance.
>
> Following tests are not ported from NotificationViewTest, mostly
> because the feature is unimplemented or the test is not applicable.
> * CreateOrUpdateTestSettingsButton
> * TestLineLimits
> * TestImageSizing
> * SettingsButtonTest
> * ViewOrderingTest
> * FormatContextMessageTest
>
> BUG=733948
> TEST=out/Debug/message_center_unittests
>
> Review-Url: https://codereview.chromium.org/2966343002
> Cr-Commit-Position: refs/heads/master@{#484851}
> Committed: https://chromium.googlesource.com/chromium/src/+/9a4a1121fae26a1727728888a41975bc15e3e62f
TBR=fukino@chromium.org,yoshiki@chromium.org,tetsui@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=733948
Review-Url: https://codereview.chromium.org/2965353002
Cr-Commit-Position: refs/heads/master@{#484899}
Committed: https://chromium.googlesource.com/chromium/src/+/a0e04c835cae01af85b8532022cec6ee5346f69f
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|