Chromium Code Reviews| Index: client/html/src/DocumentFragmentWrappingImplementation.dart | 
| =================================================================== | 
| --- client/html/src/DocumentFragmentWrappingImplementation.dart (revision 942) | 
| +++ client/html/src/DocumentFragmentWrappingImplementation.dart (working copy) | 
| @@ -99,10 +99,14 @@ | 
| Iterator<Element> iterator() => _filtered.iterator(); | 
| List<Element> getRange(int start, int length) => | 
| _filtered.getRange(start, length); | 
| - int indexOf(Element element, int startIndex) => | 
| - _filtered.indexOf(element, startIndex); | 
| - int lastIndexOf(Element element, int startIndex) => | 
| - _filtered.lastIndexOf(element, startIndex); | 
| + int indexOf(Element element, [int start = 0]) => | 
| + _filtered.indexOf(element, start); | 
| + | 
| + int lastIndexOf(Element element, [int start = null]) { | 
| 
 
srdjan
2011/10/31 18:01:29
WHy not just [int start] (here and everywhere else
 
ngeoffray
2011/11/01 07:50:44
Last time I checked, not giving a default value fa
 
 | 
| + if (start === null) start = length - 1; | 
| + return _filtered.lastIndexOf(element, start); | 
| + } | 
| + | 
| Element last() => _filtered.last(); | 
| } | 
| @@ -362,4 +366,4 @@ | 
| throw new UnsupportedOperationException( | 
| "WebKit drop zone can't be set for document fragments."); | 
| } | 
| -} | 
| +} |