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

Issue 19965003: Fix sublist check, add missing methods and add tests. (Closed)

Created:
7 years, 5 months ago by floitsch
Modified:
7 years, 5 months ago
Reviewers:
ngeoffray
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fix sublist check, add missing methods and add tests for typed data lists. Somehow these lists were still missing functions (in the VM) and the sublist check was broken (in dart2js). R=ngeoffray@google.com Committed: https://code.google.com/p/dart/source/detail?r=25333

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+199 lines, -3 lines) Patch
M runtime/lib/typed_data.dart View 2 chunks +8 lines, -0 lines 0 comments Download
M sdk/lib/typed_data/dart2js/typed_data_dart2js.dart View 1 chunk +7 lines, -3 lines 0 comments Download
A tests/lib/typed_data/typed_data_list_test.dart View 1 chunk +184 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
floitsch
7 years, 5 months ago (2013-07-22 16:05:09 UTC) #1
ngeoffray
LGTM!
7 years, 5 months ago (2013-07-23 09:17:37 UTC) #2
floitsch
7 years, 5 months ago (2013-07-23 09:20:37 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r25333 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698