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

Issue 11377025: Mark pkg/http/test/request_test flaky on Windows (Closed)

Created:
8 years, 1 month ago by Søren Gjesse
Modified:
8 years, 1 month ago
Reviewers:
ahe, Mads Ager (google)
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Mark pkg/http/test/request_test flaky on Windows R=ager@google.com BUG=dart:6594 Committed: https://code.google.com/p/dart/source/detail?r=14624

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M pkg/pkg.status View 1 chunk +2 lines, -1 line 2 comments Download

Messages

Total messages: 4 (0 generated)
Søren Gjesse
8 years, 1 month ago (2012-11-07 14:24:42 UTC) #1
Mads Ager (google)
LGTM, thanks
8 years, 1 month ago (2012-11-07 14:25:09 UTC) #2
ahe
https://codereview.chromium.org/11377025/diff/1/pkg/pkg.status File pkg/pkg.status (right): https://codereview.chromium.org/11377025/diff/1/pkg/pkg.status#newcode59 pkg/pkg.status:59: pkg/http/test/request_test: Pass, Timeout # Issue 6594 I think you ...
8 years, 1 month ago (2012-11-07 15:21:29 UTC) #3
Søren Gjesse
8 years, 1 month ago (2012-11-07 15:25:17 UTC) #4
https://codereview.chromium.org/11377025/diff/1/pkg/pkg.status
File pkg/pkg.status (right):

https://codereview.chromium.org/11377025/diff/1/pkg/pkg.status#newcode59
pkg/pkg.status:59: pkg/http/test/request_test: Pass, Timeout # Issue 6594
On 2012/11/07 15:21:30, ahe wrote:
> I think you need to remove pkg/.

Thanks for spotting this. Fixed in https://codereview.chromium.org/11364125/.

Powered by Google App Engine
This is Rietveld 408576698