Disable the flaky RestartTest.PRE_LocalStorageClearedOnExit on Mac

Disable the flaky RestartTest.PRE_LocalStorageClearedOnExit on Mac

TBR=sky@chromium.org
BUG=717740
Review-Url: https://codereview.chromium.org/2858683003
Cr-Original-Commit-Position: refs/heads/master@{#468963}
Committed: https://chromium.googlesource.com/chromium/src/+/dd799306dbacf94b73b935d9870d0119f06c2304
Review-Url: https://codereview.chromium.org/2858683003
Cr-Commit-Position: refs/heads/master@{#469021}
Committed: https://chromium.googlesource.com/chromium/src/+/666f9f44c4e0aa571a167f7b9b9a8c15a6a0e964

[msramek, 2017-05-03 14:29:37]

Description was changed from

==========
Disable the flaky RestartTest.PRE_LocalStorageClearedOnExit on Mac

TBR=sky@chromium.org
BUG=717740
==========

to

==========
Disable the flaky RestartTest.PRE_LocalStorageClearedOnExit on Mac

TBR=sky@chromium.org
NOTRY=True
BUG=717740
==========

[msramek, 2017-05-03 14:30:09]

Sheriff here, disabling a flaky test. sky@, I'm adding you to TBR as the only
owner of the file.

[msramek, 2017-05-03 14:30:14]

The CQ bit was checked by msramek@chromium.org

[commit-bot: I haz the power, 2017-05-03 14:30:31]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-03 14:35:52]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493821814865730, "parent_rev":
"d3c675e53e8b9c17d77e47107d04df8d0baae069", "commit_rev":
"dd799306dbacf94b73b935d9870d0119f06c2304"}

[commit-bot: I haz the power, 2017-05-03 14:35:59]

Description was changed from

==========
Disable the flaky RestartTest.PRE_LocalStorageClearedOnExit on Mac

TBR=sky@chromium.org
NOTRY=True
BUG=717740
==========

to

==========
Disable the flaky RestartTest.PRE_LocalStorageClearedOnExit on Mac

TBR=sky@chromium.org
NOTRY=True
BUG=717740

Review-Url: https://codereview.chromium.org/2858683003
Cr-Commit-Position: refs/heads/master@{#468963}
Committed:
https://chromium.googlesource.com/chromium/src/+/dd799306dbacf94b73b935d9870d...
==========

[commit-bot: I haz the power, 2017-05-03 14:36:00]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/dd799306dbacf94b73b935d9870d...

[msramek, 2017-05-03 14:50:27]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2862553003/ by msramek@chromium.org.

The reason for reverting is: Sigh... a typo. Reverting..

[msramek, 2017-05-03 14:54:12]

Description was changed from

==========
Disable the flaky RestartTest.PRE_LocalStorageClearedOnExit on Mac

TBR=sky@chromium.org
NOTRY=True
BUG=717740

Review-Url: https://codereview.chromium.org/2858683003
Cr-Commit-Position: refs/heads/master@{#468963}
Committed:
https://chromium.googlesource.com/chromium/src/+/dd799306dbacf94b73b935d9870d...
==========

to

==========
Disable the flaky RestartTest.PRE_LocalStorageClearedOnExit on Mac

TBR=sky@chromium.org
BUG=717740

Review-Url: https://codereview.chromium.org/2858683003
Cr-Commit-Position: refs/heads/master@{#468963}
Committed:
https://chromium.googlesource.com/chromium/src/+/dd799306dbacf94b73b935d9870d...
==========

[msramek, 2017-05-03 14:55:20]

Sorry for the noise. I thought I couldn't make a typo in such a simple CL, but I
did.

Fixed and landing again, this time through the trybots to make sure.

[msramek, 2017-05-03 14:55:38]

The CQ bit was checked by msramek@chromium.org

[commit-bot: I haz the power, 2017-05-03 14:55:59]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[findit-for-me, 2017-05-03 15:00:26]

Findit (https://goo.gl/kROfz5) confirmed this CL at revision 468963 as the
culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...

[commit-bot: I haz the power, 2017-05-03 15:34:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-03 15:34:46]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[msramek, 2017-05-03 15:42:01]

The CQ bit was checked by msramek@chromium.org

[commit-bot: I haz the power, 2017-05-03 15:42:24]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-03 16:25:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-03 16:25:02]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[msramek, 2017-05-03 16:26:43]

The CQ bit was checked by msramek@chromium.org

[commit-bot: I haz the power, 2017-05-03 16:27:16]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[msramek, 2017-05-03 16:33:05]

Patchset #3 (id:40001) has been deleted

[msramek, 2017-05-03 16:34:46]

Upon closer look, I need to disable not only
RestartTest.PRE_LocalStorageClearedOnExit, but also the actual
RestartTest.LocalStorageClearedOnExit.

Sorry for more noise, I haven't seen this construct before.

[msramek, 2017-05-03 16:34:50]

The CQ bit was checked by msramek@chromium.org

[commit-bot: I haz the power, 2017-05-03 16:35:28]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-03 17:31:11]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1493829290317110,
"parent_rev": "d8478a85a577aa1ac3d1bfb1a6f1d81e3de9e385", "commit_rev":
"666f9f44c4e0aa571a167f7b9b9a8c15a6a0e964"}

[commit-bot: I haz the power, 2017-05-03 17:31:20]

Description was changed from

==========
Disable the flaky RestartTest.PRE_LocalStorageClearedOnExit on Mac

TBR=sky@chromium.org
BUG=717740

Review-Url: https://codereview.chromium.org/2858683003
Cr-Commit-Position: refs/heads/master@{#468963}
Committed:
https://chromium.googlesource.com/chromium/src/+/dd799306dbacf94b73b935d9870d...
==========

to

==========
Disable the flaky RestartTest.PRE_LocalStorageClearedOnExit on Mac

TBR=sky@chromium.org
BUG=717740

Review-Url: https://codereview.chromium.org/2858683003
Cr-Original-Commit-Position: refs/heads/master@{#468963}
Committed:
https://chromium.googlesource.com/chromium/src/+/dd799306dbacf94b73b935d9870d...
Review-Url: https://codereview.chromium.org/2858683003
Cr-Commit-Position: refs/heads/master@{#469021}
Committed:
https://chromium.googlesource.com/chromium/src/+/666f9f44c4e0aa571a167f7b9b9a...
==========

[commit-bot: I haz the power, 2017-05-03 17:31:22]

Committed patchset #3 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/666f9f44c4e0aa571a167f7b9b9a...

[msramek, 2017-05-03 18:30:51]

A revert of this CL (patchset #3 id:60001) has been created in
https://codereview.chromium.org/2862583003/ by msramek@chromium.org.

The reason for reverting is: And reverting again, because MAYBE_ and PRE_
directives cannot be used together..