Chromium Code Reviews| Index: sdk/lib/_internal/compiler/implementation/lib/js_array.dart |
| diff --git a/sdk/lib/_internal/compiler/implementation/lib/js_array.dart b/sdk/lib/_internal/compiler/implementation/lib/js_array.dart |
| index 4bd608853c589ced5b44801fd8c8cb9e8159c5af..56f3eacecbd66c925001b864c1f8dd3c5e8f6198 100644 |
| --- a/sdk/lib/_internal/compiler/implementation/lib/js_array.dart |
| +++ b/sdk/lib/_internal/compiler/implementation/lib/js_array.dart |
| @@ -84,10 +84,12 @@ class JSArray<E> implements List<E> { |
| return IterableMixinWorkaround.forEach(this, f); |
| } |
| - List mappedBy(f(E element)) { |
| + Iterable map(f(E element)) { |
| return IterableMixinWorkaround.mappedByList(this, f); |
|
floitsch
2013/01/30 14:48:44
should return an Iterable.
Lasse Reichstein Nielsen
2013/01/31 11:33:49
Done.
|
| } |
| + Iterable mappedBy(f(E element)) => map(f); |
| + |
| String join([String separator]) { |
| if (separator == null) separator = ""; |
| var list = new List(this.length); |
| @@ -97,7 +99,7 @@ class JSArray<E> implements List<E> { |
| return JS('String', "#.join(#)", list, separator); |
| } |
| - List<E> take(int n) { |
| + Iterable<E> take(int n) { |
| return IterableMixinWorkaround.takeList(this, n); |
| } |
| @@ -105,7 +107,7 @@ class JSArray<E> implements List<E> { |
| return IterableMixinWorkaround.takeWhile(this, test); |
| } |
| - List<E> skip(int n) { |
| + Iterable<E> skip(int n) { |
| return IterableMixinWorkaround.skipList(this, n); |
| } |