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

Issue 8343038: Mark Worker ui layout tests flaky. (Closed)

Created:
9 years, 1 month ago by dmichael (off chromium)
Modified:
9 years, 1 month ago
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, Paweł Hajdan Jr., jam, dpranke-watch+content_chromium.org
Visibility:
Public.

Description

Mark Worker ui layout tests flaky. BUG=101996 TEST=N/A Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107639

Patch Set 1 #

Patch Set 2 : Use MAYBE_ pattern as suggested. #

Patch Set 3 : Do it right this time! #

Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -7 lines) Patch
M content/worker/test/worker_uitest.cc View 1 2 3 chunks +31 lines, -7 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
dmichael(do not use this one)
9 years, 1 month ago (2011-10-27 22:15:30 UTC) #1
rohitrao (ping after 24h)
Do other flaky tests use this pattern? I'm used to seeing MAYBE_TestName, where MAYBE_TestName is ...
9 years, 1 month ago (2011-10-27 22:17:26 UTC) #2
eroman
lgtm
9 years, 1 month ago (2011-10-27 22:17:41 UTC) #3
eroman
Actually yeah, what rohitrao said. I usually see this as MAYBE_
9 years, 1 month ago (2011-10-27 22:18:49 UTC) #4
dmichael (off chromium)
Okay, I'd mostly seen the other style before (this file has a little of both), ...
9 years, 1 month ago (2011-10-27 22:29:16 UTC) #5
dmichael (off chromium)
one more try (?)
9 years, 1 month ago (2011-10-27 22:36:35 UTC) #6
rohitrao (ping after 24h)
9 years, 1 month ago (2011-10-27 22:38:16 UTC) #7
LGTM

Please commit this directly with --force instead of sending it through the
commit queue.

Powered by Google App Engine
This is Rietveld 408576698