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

Issue 9111012: Detect crashes in frog/leg (when running on VM). (Closed)

Created:
8 years, 11 months ago by ahe
Modified:
8 years, 11 months ago
Reviewers:
ngeoffray, kasperl
CC:
reviews_dartlang.org, Lasse Reichstein Nielsen, floitsch, karlklose
Base URL:
ssh://aaricia.aar/home/ahe/Dart/all/dart.googlecode.com.git@master
Visibility:
Public.

Description

Detect crashes in frog/leg (when running on VM). Committed: https://code.google.com/p/dart/source/detail?r=2992

Patch Set 1 #

Total comments: 5

Patch Set 2 : Fix mistake in status file #

Unified diffs Side-by-side diffs Delta from patch set Stats (+153 lines, -75 lines) Patch
M dart/frog/frogc.dart View 1 chunk +14 lines, -1 line 0 comments Download
M dart/frog/minfrogc.dart View 1 chunk +2 lines, -1 line 0 comments Download
M dart/frog/scripts/bootstrap/frog_wrapper.py View 1 chunk +5 lines, -1 line 0 comments Download
M dart/frog/tests/leg_only/leg_only.status View 1 chunk +6 lines, -0 lines 0 comments Download
M dart/tests/corelib/corelib.status View 3 chunks +10 lines, -2 lines 0 comments Download
M dart/tests/corelib/corelib-leg.status View 3 chunks +19 lines, -19 lines 0 comments Download
M dart/tests/isolate/isolate.status View 1 1 chunk +5 lines, -0 lines 0 comments Download
M dart/tests/language/language.status View 2 chunks +12 lines, -2 lines 0 comments Download
M dart/tests/language/language-leg.status View 18 chunks +80 lines, -49 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
ahe
I may have to fiddle a little more with the status files before this CL ...
8 years, 11 months ago (2012-01-05 11:50:50 UTC) #1
ahe
In case it isn't obvious: frogc.dart and minfrogc.dart are Dart VM wrappers around Frog. So ...
8 years, 11 months ago (2012-01-05 11:56:18 UTC) #2
ngeoffray
LGTM, but http://codereview.chromium.org/9111012/diff/1/dart/tests/isolate/isolate.status File dart/tests/isolate/isolate.status (right): http://codereview.chromium.org/9111012/diff/1/dart/tests/isolate/isolate.status#newcode36 dart/tests/isolate/isolate.status:36: [ $component == frogium ] On 2012/01/05 ...
8 years, 11 months ago (2012-01-05 12:09:19 UTC) #3
ahe
8 years, 11 months ago (2012-01-05 12:14:44 UTC) #4
http://codereview.chromium.org/9111012/diff/1/dart/tests/isolate/isolate.status
File dart/tests/isolate/isolate.status (right):

http://codereview.chromium.org/9111012/diff/1/dart/tests/isolate/isolate.stat...
dart/tests/isolate/isolate.status:36: [ $component == frogium ]
On 2012/01/05 12:09:19, ngeoffray wrote:
> On 2012/01/05 11:56:18, ahe wrote:
> > For example, this should be frog, not frogium :-)
> 
> Please explain the comment. I don't understand why frogium (which does not run
> on the VM) would now crash.

This comment follows up to my first message where I said that I might need to
fiddle more with the status file (but I can see that context got completely
lost).

I made a mistake: it should be "frog", not "frogium" here. I have fixed that
locally, but not uploaded a new patch set yet.

http://codereview.chromium.org/9111012/diff/1/dart/tests/language/language.st...
File dart/tests/language/language.status (right):

http://codereview.chromium.org/9111012/diff/1/dart/tests/language/language.st...
dart/tests/language/language.status:347: [ $component == frogsh || $component ==
frogium ]
On 2012/01/05 12:09:19, ngeoffray wrote:
> Again, why are frogsh and frogium affected?

They aren't. I have moved these tests from a frog+frogsh+frogium section to a
frogsh+frogium section.

Powered by Google App Engine
This is Rietveld 408576698