Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(297)

Issue 2873039: Mark PauseWhenNotConnected as flaky as it keeps timing out,... (Closed)

Created:
10 years, 5 months ago by tim (not reviewing)
Modified:
9 years, 6 months ago
CC:
chromium-reviews, ncarter (slow), ben+cc_chromium.org, Raghu Simha, idana, tim (not reviewing)
Visibility:
Public.

Description

Mark PauseWhenNotConnected as flaky as it keeps timing out, and ServerDeletingFolderWeHaveAnOpenEntryIn is failing 100% of the time. BUG=39070, 46621 TEST=Syncer unittest, SyncerThreadTest TBR=skrul Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=51505

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Patch
M chrome/browser/sync/engine/syncer_thread_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/sync/engine/syncer_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
tim (not reviewing)
10 years, 5 months ago (2010-07-02 06:16:49 UTC) #1
Paweł Hajdan Jr.
Drive-by. Timing out = DISABLED, not FLAKY. Failing 100% = FAILS, not DISABLED. Sorry that ...
10 years, 5 months ago (2010-07-02 06:35:42 UTC) #2
tim (not reviewing)
What determines what turns the bots orange / what output shows up in the waterfall? ...
10 years, 5 months ago (2010-07-02 06:44:23 UTC) #3
Paweł Hajdan Jr.
Yes, FAILS turns the bot orange and not red. On Fri, Jul 2, 2010 at ...
10 years, 5 months ago (2010-07-02 07:22:00 UTC) #4
skrul
LGTM
10 years, 5 months ago (2010-07-07 16:36:24 UTC) #5
tim (not reviewing)
10 years, 5 months ago (2010-07-07 17:03:27 UTC) #6
So turns out I was misinterpreting DISABLED and FLAKY, I'll whip up a patch
to make it as Pawel suggests~

On Wed, Jul 7, 2010 at 9:36 AM, <skrul@chromium.org> wrote:

> LGTM
>
>
> http://codereview.chromium.org/2873039/show
>

Powered by Google App Engine
This is Rietveld 408576698