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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 841193003: cleanup to tools/testing/dart (Closed)

Created:
5 years, 11 months ago by kevmoo
Modified:
5 years, 11 months ago
Reviewers:
ricow1
CC:
reviews_dartlang.org
Target Ref:
refs/remotes/git-svn
Visibility:
Public.

Description

cleanup Dart test code

Patch Set 1 #

Patch Set 2 : more updates #

Patch Set 3 : roll back a tweak #

Patch Set 4 : another tweak #

Patch Set 5 : one last bit #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1268 lines, -9632 lines) Patch
M tests/standalone/io/dependency_graph_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M tests/standalone/io/skipping_dart2js_compilations_test.dart View 1 6 chunks +16 lines, -15 lines 0 comments Download
M tests/standalone/io/status_file_parser_test.dart View 1 chunk +5 lines, -4 lines 0 comments Download
M tests/standalone/io/test_harness_analyzer_test.dart View 1 1 chunk +1 line, -1 line 0 comments Download
M tests/standalone/io/test_runner_test.dart View 1 1 chunk +5 lines, -3 lines 0 comments Download
M tests/standalone/status_expression_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M tools/status_clean.dart View 1 chunk +4 lines, -3 lines 0 comments Download
M tools/test.dart View 1 1 chunk +3 lines, -3 lines 0 comments Download
M tools/testing/dart/android.dart View 1 1 chunk +0 lines, -334 lines 0 comments Download
M tools/testing/dart/browser_controller.dart View 1 1 chunk +0 lines, -1794 lines 0 comments Download
M tools/testing/dart/browser_test.dart View 1 1 chunk +0 lines, -51 lines 0 comments Download
M tools/testing/dart/co19_test.dart View 1 chunk +4 lines, -4 lines 0 comments Download
M tools/testing/dart/co19_test_config.dart View 1 chunk +1 line, -1 line 0 comments Download
M tools/testing/dart/compiler_configuration.dart View 1 2 3 4 1 chunk +4 lines, -23 lines 0 comments Download
D tools/testing/dart/dependency_graph.dart View 1 chunk +0 lines, -147 lines 0 comments Download
D tools/testing/dart/drt_updater.dart View 1 chunk +0 lines, -88 lines 0 comments Download
M tools/testing/dart/html_test.dart View 1 1 chunk +0 lines, -77 lines 0 comments Download
M tools/testing/dart/http_server.dart View 1 1 chunk +0 lines, -465 lines 0 comments Download
M tools/testing/dart/launch_browser.dart View 1 1 chunk +0 lines, -39 lines 0 comments Download
A + tools/testing/dart/lib/android.dart View 1 1 chunk +6 lines, -6 lines 0 comments Download
A + tools/testing/dart/lib/browser_controller.dart View 1 1 chunk +4 lines, -4 lines 0 comments Download
A + tools/testing/dart/lib/browser_test.dart View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + tools/testing/dart/lib/command.dart View 1 5 chunks +387 lines, -1805 lines 0 comments Download
A + tools/testing/dart/lib/dependency_graph.dart View 1 chunk +1 line, -1 line 0 comments Download
A + tools/testing/dart/lib/drt_updater.dart View 1 chunk +3 lines, -3 lines 0 comments Download
A + tools/testing/dart/lib/html_test.dart View 1 1 chunk +5 lines, -5 lines 0 comments Download
A + tools/testing/dart/lib/http_server.dart View 1 1 chunk +3 lines, -7 lines 0 comments Download
A + tools/testing/dart/lib/launch_browser.dart View 1 1 chunk +2 lines, -2 lines 0 comments Download
A + tools/testing/dart/lib/multitest.dart View 1 chunk +5 lines, -5 lines 0 comments Download
A + tools/testing/dart/lib/path.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tools/testing/dart/lib/record_and_replay.dart View 1 1 chunk +2 lines, -2 lines 0 comments Download
A + tools/testing/dart/lib/status_expression.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tools/testing/dart/lib/status_file_parser.dart View 1 1 chunk +5 lines, -5 lines 0 comments Download
A tools/testing/dart/lib/summary_report.dart View 1 1 chunk +106 lines, -0 lines 0 comments Download
A tools/testing/dart/lib/test_case.dart View 1 1 chunk +177 lines, -0 lines 0 comments Download
A tools/testing/dart/lib/test_information.dart View 1 chunk +33 lines, -0 lines 0 comments Download
A + tools/testing/dart/lib/test_progress.dart View 1 1 chunk +12 lines, -9 lines 0 comments Download
A tools/testing/dart/lib/test_utils.dart View 1 chunk +395 lines, -0 lines 0 comments Download
A + tools/testing/dart/lib/utils.dart View 1 chunk +11 lines, -0 lines 0 comments Download
D tools/testing/dart/multitest.dart View 1 chunk +0 lines, -307 lines 0 comments Download
D tools/testing/dart/path.dart View 1 chunk +0 lines, -317 lines 0 comments Download
M tools/testing/dart/record_and_replay.dart View 1 1 chunk +0 lines, -136 lines 0 comments Download
M tools/testing/dart/runtime_configuration.dart View 1 2 3 4 1 chunk +14 lines, -4 lines 0 comments Download
D tools/testing/dart/status_expression.dart View 1 chunk +0 lines, -330 lines 0 comments Download
D tools/testing/dart/status_file_parser.dart View 1 chunk +0 lines, -352 lines 0 comments Download
M tools/testing/dart/test_configurations.dart View 1 1 chunk +12 lines, -11 lines 0 comments Download
M tools/testing/dart/test_options.dart View 1 1 chunk +7 lines, -6 lines 0 comments Download
M tools/testing/dart/test_progress.dart View 1 1 chunk +0 lines, -670 lines 0 comments Download
M tools/testing/dart/test_runner.dart View 1 3 chunks +17 lines, -1710 lines 0 comments Download
M tools/testing/dart/test_suite.dart View 1 2 3 4 6 chunks +18 lines, -557 lines 0 comments Download
D tools/testing/dart/utils.dart View 1 chunk +0 lines, -310 lines 0 comments Download
D tools/testing/dart/vm_test_config.dart View 1 1 chunk +0 lines, -16 lines 0 comments Download
A + tools/testing/dart/vm_test_suite.dart View 1 2 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (1 generated)
kevmoo
5 years, 11 months ago (2015-01-09 00:58:31 UTC) #2
ricow1
why are you moving these?
5 years, 11 months ago (2015-01-09 08:14:30 UTC) #3
kevmoo
5 years, 11 months ago (2015-01-12 18:42:30 UTC) #4 
On 2015/01/09 08:14:30, ricow1 wrote:
> why are you moving these?

I'm killing this CL. Got a bit excited refactoring. See latest CL

Powered by Google App Engine
This is Rietveld 408576698