DescriptionReland of Don't allow a widget to send close notifications more than once. (patchset #1 id:1 of https://codereview.chromium.org/2844933002/ )
Reason for revert:
We will disable the failing test on Mac.
Original issue's description:
> Revert of Don't allow a widget to send close notifications more than once. (patchset #3 id:40001 of https://codereview.chromium.org/2834943002/ )
>
> Reason for revert:
> The enabled WidgetObserverTest.ActivationChange test seems to be flaky
>
> https://build.chromium.org/p/chromium.mac/builders/Mac10.10%20Tests/builds/17210
>
> Original issue's description:
> > Don't allow a widget to send close notifications more than once.
> >
> > Add a test to make sure this works. Also fix a couple WidgetTests that
> > were disabled.
> >
> > BUG=714334
> >
> > Review-Url: https://codereview.chromium.org/2834943002
> > Cr-Commit-Position: refs/heads/master@{#467171}
> > Committed: https://chromium.googlesource.com/chromium/src/+/bf25b17bd99b5dbeff92e62f8dddd289c335dc0a
>
> TBR=sky@chromium.org,estade@chromium.org
> NOTRY=true
> BUG=714334
>
> Review-Url: https://codereview.chromium.org/2844933002
> Cr-Commit-Position: refs/heads/master@{#467528}
> Committed: https://chromium.googlesource.com/chromium/src/+/efd8268f95f1c7a212e5b6edd521761dd8a00e31
TBR=sky@chromium.org,alph@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=714334
Review-Url: https://codereview.chromium.org/2839403002
Cr-Commit-Position: refs/heads/master@{#467544}
Committed: https://chromium.googlesource.com/chromium/src/+/3a303320be089d790c88890ffc28617415238c5c
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|