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

Issue 17590006: Change test to use JSExtendableArray instead of =List. (Closed)

Created:
7 years, 6 months ago by ngeoffray
Modified:
7 years, 6 months ago
Reviewers:
polux, kasperl
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Change test to use JSExtendableArray instead of =List. Committed: https://code.google.com/p/dart/source/detail?r=24322

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -3 lines) Patch
M tests/compiler/dart2js/cpa_inference_test.dart View 4 chunks +7 lines, -3 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
ngeoffray
@Paul: I could not figure out why the inferred type is now {null} instead of ...
7 years, 6 months ago (2013-06-24 10:46:15 UTC) #1
polux
On 2013/06/24 10:46:15, ngeoffray wrote: > @Paul: I could not figure out why the inferred ...
7 years, 6 months ago (2013-06-24 10:49:11 UTC) #2
polux
On 2013/06/24 10:49:11, polux wrote: > On 2013/06/24 10:46:15, ngeoffray wrote: > > @Paul: I ...
7 years, 6 months ago (2013-06-24 10:51:19 UTC) #3
ngeoffray
On 2013/06/24 10:51:19, polux wrote: > On 2013/06/24 10:49:11, polux wrote: > > On 2013/06/24 ...
7 years, 6 months ago (2013-06-24 10:51:55 UTC) #4
polux
On 2013/06/24 10:51:55, ngeoffray wrote: > On 2013/06/24 10:51:19, polux wrote: > > On 2013/06/24 ...
7 years, 6 months ago (2013-06-24 10:54:10 UTC) #5
ngeoffray
7 years, 6 months ago (2013-06-24 10:54:41 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 manually as r24322 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698