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

Issue 11361031: Update html status file. (Closed)

Created:
8 years, 1 month ago by ngeoffray
Modified:
8 years, 1 month ago
Reviewers:
kasperl
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Update html status file. Committed: https://code.google.com/p/dart/source/detail?r=14390

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M tests/html/html.status View 1 chunk +1 line, -1 line 2 comments Download

Messages

Total messages: 3 (0 generated)
ngeoffray
TBR
8 years, 1 month ago (2012-11-01 10:11:24 UTC) #1
kasperl
LGTM. https://codereview.chromium.org/11361031/diff/1/tests/html/html.status File tests/html/html.status (right): https://codereview.chromium.org/11361031/diff/1/tests/html/html.status#newcode182 tests/html/html.status:182: node_test: Fail, Crash Did you file a bug ...
8 years, 1 month ago (2012-11-01 10:15:48 UTC) #2
ngeoffray
8 years, 1 month ago (2012-11-01 10:36:58 UTC) #3
https://codereview.chromium.org/11361031/diff/1/tests/html/html.status
File tests/html/html.status (right):

https://codereview.chromium.org/11361031/diff/1/tests/html/html.status#newcod...
tests/html/html.status:182: node_test: Fail, Crash
On 2012/11/01 10:15:48, kasperl wrote:
> Did you file a bug with priority critical about this? You should.

Done: http://code.google.com/p/dart/issues/detail?id=6457

Powered by Google App Engine
This is Rietveld 408576698