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

Issue 7205001: Mark TouchGetInfoPlatformFile as Flaky on Mac OS X. (Closed)

Created:
9 years, 6 months ago by Leandro Graciá Gil
Modified:
9 years, 6 months ago
CC:
chromium-reviews, brettw-cc_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

Mark TouchGetInfoPlatformFile as Flaky on Mac OS X. BUG=86494 TEST=PlatformFile.TouchGetInfoPlatformFile Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=89479

Patch Set 1 #

Total comments: 6

Patch Set 2 : review fixes. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -1 line) Patch
M base/platform_file_unittest.cc View 1 1 chunk +7 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Leandro Gracia Gil
9 years, 6 months ago (2011-06-17 14:06:04 UTC) #1
joth
Almost :-) http://codereview.chromium.org/7205001/diff/1/base/platform_file_unittest.cc File base/platform_file_unittest.cc (right): http://codereview.chromium.org/7205001/diff/1/base/platform_file_unittest.cc#newcode255 base/platform_file_unittest.cc:255: FLAKY_TouchGetInfoPlatformFile this will go on one line ...
9 years, 6 months ago (2011-06-17 14:08:27 UTC) #2
Leandro Gracia Gil
http://codereview.chromium.org/7205001/diff/1/base/platform_file_unittest.cc File base/platform_file_unittest.cc (right): http://codereview.chromium.org/7205001/diff/1/base/platform_file_unittest.cc#newcode255 base/platform_file_unittest.cc:255: FLAKY_TouchGetInfoPlatformFile On 2011/06/17 14:08:27, joth wrote: > this will ...
9 years, 6 months ago (2011-06-17 14:12:44 UTC) #3
joth
9 years, 6 months ago (2011-06-17 14:14:20 UTC) #4
LGTM

feel free to use commit queue or land manually as you prefer

Powered by Google App Engine
This is Rietveld 408576698