Mark SyncTest#testStopAndStartSyncThroughAndroid as flaky

Mark SyncTest#testStopAndStartSyncThroughAndroid as flaky

BUG=594558
Committed: https://crrev.com/a7f875ca7a0949cbbaafa9aed15562efc3c5b80c
Cr-Commit-Position: refs/heads/master@{#380985}

[May, 2016-03-14 14:55:31]

maybelle@chromium.org changed reviewers:
+ bauerb@chromium.org

[Bernhard Bauer, 2016-03-14 15:23:50]

LGTM with nits:

https://codereview.chromium.org/1801733002/diff/1/chrome/android/sync_shell/j...
File
chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java
(right):

https://codereview.chromium.org/1801733002/diff/1/chrome/android/sync_shell/j...
chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java:124:
/*@LargeTest
Break after opening the comment, and add asterisks:

/*
 * ...
 * ...
 */

https://codereview.chromium.org/1801733002/diff/1/chrome/android/sync_shell/j...
chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java:126:
BUG = 594558
crbug.com/594558

[May, 2016-03-14 15:27:02]

The CQ bit was checked by maybelle@chromium.org

[May, 2016-03-14 15:27:02]

The patchset sent to the CQ was uploaded after l-g-t-m from bauerb@chromium.org
Link to the patchset: https://codereview.chromium.org/1801733002/#ps40001
(title: "Fix comment asterisks")

[commit-bot: I haz the power, 2016-03-14 15:27:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1801733002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1801733002/40001

[Bernhard Bauer, 2016-03-14 15:30:50]

https://codereview.chromium.org/1801733002/diff/40001/chrome/android/sync_she...
File
chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java
(right):

https://codereview.chromium.org/1801733002/diff/40001/chrome/android/sync_she...
chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java:127:
* BUG = 594558
Ahem... crbug.com/594558 pls ☺

[May, 2016-03-14 15:39:56]

On 2016/03/14 15:30:50, bauerb catching up on reviews wrote:
>
https://codereview.chromium.org/1801733002/diff/40001/chrome/android/sync_she...
> File
>
chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java
> (right):
> 
>
https://codereview.chromium.org/1801733002/diff/40001/chrome/android/sync_she...
>
chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java:127:
> * BUG = 594558
> Ahem... crbug.com/594558 pls ☺

Ack, ok -- fixed http://dev.chromium.org/developers/testing/android-tests also.

[May, 2016-03-14 15:44:01]

The CQ bit was checked by maybelle@chromium.org

[May, 2016-03-14 15:44:02]

The patchset sent to the CQ was uploaded after l-g-t-m from bauerb@chromium.org
Link to the patchset: https://codereview.chromium.org/1801733002/#ps60001
(title: "Fix BUG= link")

[commit-bot: I haz the power, 2016-03-14 15:44:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1801733002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1801733002/60001

[commit-bot: I haz the power, 2016-03-14 16:18:21]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-03-14 16:19:23]

Description was changed from

==========
Mark SyncTest#testStopAndStartSyncThroughAndroid as flaky

BUG=594558
==========

to

==========
Mark SyncTest#testStopAndStartSyncThroughAndroid as flaky

BUG=594558

Committed: https://crrev.com/a7f875ca7a0949cbbaafa9aed15562efc3c5b80c
Cr-Commit-Position: refs/heads/master@{#380985}
==========

[commit-bot: I haz the power, 2016-03-14 16:19:25]

Patchset 4 (id:??) landed as
https://crrev.com/a7f875ca7a0949cbbaafa9aed15562efc3c5b80c
Cr-Commit-Position: refs/heads/master@{#380985}