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

Issue 345053003: Fix typo (Closed)

Created:
6 years, 6 months ago by ahe
Modified:
6 years, 6 months ago
Reviewers:
vsm
CC:
reviews_dartlang.org, Bill Hesse
Visibility:
Public.

Description

Patch Set 1 #

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

Messages

Total messages: 4 (0 generated)
ahe
TBR
6 years, 6 months ago (2014-06-20 12:41:40 UTC) #1
ahe
Committed patchset #1 manually as r37544 (presubmit successful).
6 years, 6 months ago (2014-06-20 12:41:59 UTC) #2
vsm
https://codereview.chromium.org/345053003/diff/1/dart/tests/utils/utils.status File dart/tests/utils/utils.status (right): https://codereview.chromium.org/345053003/diff/1/dart/tests/utils/utils.status#newcode24 dart/tests/utils/utils.status:24: [ $compiler == none && ( $runtime == drt ...
6 years, 6 months ago (2014-06-20 12:46:11 UTC) #3
ahe
6 years, 6 months ago (2014-06-20 12:49:12 UTC) #4
Message was sent while issue was closed.
Thank you.

https://codereview.chromium.org/345053003/diff/1/dart/tests/utils/utils.status
File dart/tests/utils/utils.status (right):

https://codereview.chromium.org/345053003/diff/1/dart/tests/utils/utils.statu...
dart/tests/utils/utils.status:24: [ $compiler == none && ( $runtime == drt  ||
$runtime == dartium ) ]
On 2014/06/20 12:46:11, vsm wrote:
> Actually, we probably need $runtime == ContentShellOnAndroid as well.  That
bot
> is hosed at the moment, but it looks like it normally runs that test.

https://codereview.chromium.org/343153002/

Powered by Google App Engine
This is Rietveld 408576698