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

Issue 1328213003: Fix tests that explicitly part in io library files (Closed)

Created:
5 years, 3 months ago by ricow1
Modified:
5 years, 3 months ago
Reviewers:
Søren Gjesse
CC:
reviews_dartlang.org
Base URL:
https://github.com/dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix tests that explicitly part in io library files R=sgjesse@google.com BUG= Committed: https://github.com/dart-lang/sdk/commit/c99e5c02fa68a6c583c746b34768e9f6882185b8

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -0 lines) Patch
M tests/standalone/io/http_cookie_date_test.dart View 2 chunks +2 lines, -0 lines 0 comments Download
M tests/standalone/io/http_headers_test.dart View 2 chunks +2 lines, -0 lines 0 comments Download
M tests/standalone/io/http_parser_test.dart View 2 chunks +2 lines, -0 lines 0 comments Download
M tests/standalone/io/web_socket_protocol_processor_test.dart View 2 chunks +2 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
ricow1
5 years, 3 months ago (2015-09-08 10:19:11 UTC) #1
ricow1
analyzer does not like them: http://build.chromium.org/p/client.dart/builders/analyzer_experimental-linux-release-be/builds/3426/steps/tests%20--host-checked/logs/stdio
5 years, 3 months ago (2015-09-08 10:19:41 UTC) #2
ricow1
tbr
5 years, 3 months ago (2015-09-08 11:29:04 UTC) #3
ricow1
Committed patchset #1 (id:1) manually as c99e5c02fa68a6c583c746b34768e9f6882185b8 (presubmit successful).
5 years, 3 months ago (2015-09-08 11:30:43 UTC) #4
Søren Gjesse
5 years, 3 months ago (2015-09-08 11:38:10 UTC) #5
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698