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

Issue 6127004: Disable crashing mozilla test from bug 1032 on arm with crankshaft (Closed)

Created:
9 years, 11 months ago by Rico
Modified:
9 years, 6 months ago
CC:
v8-dev
Visibility:
Public.

Description

Disable crashing mozilla test from bug 1032 on arm with crankshaft Committed: http://code.google.com/p/v8/source/detail?r=6255

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M test/mozilla/mozilla.status View 1 chunk +3 lines, -0 lines 2 comments Download

Messages

Total messages: 3 (0 generated)
Rico
9 years, 11 months ago (2011-01-11 07:36:13 UTC) #1
Søren Thygesen Gjesse
LGTM http://codereview.chromium.org/6127004/diff/1/test/mozilla/mozilla.status File test/mozilla/mozilla.status (right): http://codereview.chromium.org/6127004/diff/1/test/mozilla/mozilla.status#newcode844 test/mozilla/mozilla.status:844: mozilla/ecma/Date/15.9.3.1-1: PASS || CRASH Should this have mozilla/ ...
9 years, 11 months ago (2011-01-11 07:46:32 UTC) #2
Rico
9 years, 11 months ago (2011-01-11 07:53:25 UTC) #3
http://codereview.chromium.org/6127004/diff/1/test/mozilla/mozilla.status
File test/mozilla/mozilla.status (right):

http://codereview.chromium.org/6127004/diff/1/test/mozilla/mozilla.status#new...
test/mozilla/mozilla.status:844: mozilla/ecma/Date/15.9.3.1-1: PASS || CRASH
On 2011/01/11 07:46:32, Søren Gjesse wrote:
> Should this have mozilla/ at the beginning?
No, corrected
> 
> Maybe we should add ecma/Date/15.9.3.1-4 as well, as that crashed in r6252,
and
> the changes in r6253 and r6254 where only presubmit related.
Done

Powered by Google App Engine
This is Rietveld 408576698