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

Issue 10228007: Add handler for Process.onError in test_runner.dart. (Closed)

Created:
8 years, 8 months ago by Bill Hesse
Modified:
8 years, 8 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Add handler for Process.onError in test_runner.dart. The handler prints informaion, and does nothing. This reproduces the current behavior with no handler, and is a first step to handling this error correctly. BUG=dart:2726 TEST=tools/test.dart (Selenium and Browser tests?) Committed: https://code.google.com/p/dart/source/detail?r=6990

Patch Set 1 #

Patch Set 2 : Add bug number #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -1 line) Patch
M tools/testing/dart/test_runner.dart View 5 chunks +21 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Bill Hesse
8 years, 8 months ago (2012-04-25 15:13:47 UTC) #1
Bill Hesse
8 years, 8 months ago (2012-04-25 15:16:10 UTC) #2
Søren Gjesse
8 years, 8 months ago (2012-04-26 06:47:20 UTC) #3
lgtm

Powered by Google App Engine
This is Rietveld 408576698