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

Issue 233183002: Mark pub/test/serve/web_socket/url_to_asset_id_test slow (Closed)

Created:
6 years, 8 months ago by ricow1
Modified:
6 years, 8 months ago
Reviewers:
Bob Nystrom, kasperl
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Mark pub/test/serve/web_socket/url_to_asset_id_test slow This test has a todo to split it up into smaller components I don't know what has triggered this to go over the limit more easily R=kasperl@google.com Committed: https://code.google.com/p/dart/source/detail?r=34924

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M sdk/lib/_internal/pub/pub.status View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
ricow1
6 years, 8 months ago (2014-04-10 12:57:39 UTC) #1
kasperl
LGTM.
6 years, 8 months ago (2014-04-10 12:59:23 UTC) #2
ricow1
Committed patchset #1 manually as r34924 (presubmit successful).
6 years, 8 months ago (2014-04-10 12:59:44 UTC) #3
Bob Nystrom
6 years, 8 months ago (2014-04-10 15:50:08 UTC) #4
Message was sent while issue was closed.
LGTM, though the perf degradation is a bit alarming. I'll look into it.

Powered by Google App Engine
This is Rietveld 408576698