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

Issue 17570006: Don't test links on Windows, as we can't create relative links, in http_server. (Closed)

Created:
7 years, 6 months ago by Anders Johnsen
Modified:
7 years, 6 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Don't test links on Windows, as we can't create relative links, in http_server. BUG= Committed: https://code.google.com/p/dart/source/detail?r=24312

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+82 lines, -76 lines) Patch
M pkg/http_server/test/virtual_directory_test.dart View 3 chunks +82 lines, -76 lines 1 comment Download

Messages

Total messages: 4 (0 generated)
Anders Johnsen
7 years, 6 months ago (2013-06-24 09:07:19 UTC) #1
Anders Johnsen
Committed patchset #1 manually as r24312 (presubmit successful).
7 years, 6 months ago (2013-06-24 09:07:36 UTC) #2
Anders Johnsen
TBR
7 years, 6 months ago (2013-06-24 09:07:55 UTC) #3
Søren Gjesse
7 years, 6 months ago (2013-06-24 09:49:57 UTC) #4
Message was sent while issue was closed.
lgtm

https://codereview.chromium.org/17570006/diff/1/pkg/http_server/test/virtual_...
File pkg/http_server/test/virtual_directory_test.dart (right):

https://codereview.chromium.org/17570006/diff/1/pkg/http_server/test/virtual_...
pkg/http_server/test/virtual_directory_test.dart:138: group('links', () {
group is missing a predicate argument.

Powered by Google App Engine
This is Rietveld 408576698