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

Issue 10701078: Print even less. (Closed)

Created:
8 years, 5 months ago by Anton Muhin
Modified:
8 years, 5 months ago
Reviewers:
ricow1
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Print even less. That helped for Linux and Mac. Let's see if it'll help on Windows. R=ricow@google.com BUG=3834 Committed: https://code.google.com/p/dart/source/detail?r=9335

Patch Set 1 #

Total comments: 2

Patch Set 2 : Rebased #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -7 lines) Patch
M tests/corelib/corelib.status View 1 1 chunk +0 lines, -3 lines 0 comments Download
M tests/corelib/sort_helper.dart View 2 chunks +4 lines, -4 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
ricow1
LGTM https://chromiumcodereview.appspot.com/10701078/diff/1/tests/corelib/corelib.status File tests/corelib/corelib.status (left): https://chromiumcodereview.appspot.com/10701078/diff/1/tests/corelib/corelib.status#oldcode82 tests/corelib/corelib.status:82: list_sort_test: Fail, Pass # Issue 2667 You probably ...
8 years, 5 months ago (2012-07-03 09:36:59 UTC) #1
Anton Muhin
8 years, 5 months ago (2012-07-03 11:01:45 UTC) #2
https://chromiumcodereview.appspot.com/10701078/diff/1/tests/corelib/corelib....
File tests/corelib/corelib.status (left):

https://chromiumcodereview.appspot.com/10701078/diff/1/tests/corelib/corelib....
tests/corelib/corelib.status:82: list_sort_test: Fail, Pass # Issue 2667
Thanks, will do.

On 2012/07/03 09:36:59, ricow1 wrote:
> You probably need to rebaseline, I changed these lines this morning

Powered by Google App Engine
This is Rietveld 408576698