Add optional arguments to our indexOf/lastIndexOf methods.

client/html/src/DocumentFragmentWrappingImplementation.dart

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 class FilteredElementList implements ElementList {
   final Node _node;
   final NodeList _childNodes;
 
   FilteredElementList(Node node): _childNodes = node.nodes, _node = node;
 
@@ skipping 81 matching lines @@
 
   Collection<Element> filter(bool f(Element element)) => _filtered.filter(f);
   bool every(bool f(Element element)) => _filtered.every(f);
   bool some(bool f(Element element)) => _filtered.some(f);
   bool isEmpty() => _filtered.isEmpty();
   int get length() => _filtered.length;
   Element operator [](int index) => _filtered[index];
   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 where default value is null)?

[ngeoffray, 2011/11/01 07:50:44]

Last time I checked, not giving a default value failed on DartC.

+    if (start === null) start = length - 1;
+    return _filtered.lastIndexOf(element, start);
+  }
+
   Element last() => _filtered.last();
 }
 
 class EmptyStyleDeclaration extends CSSStyleDeclarationWrappingImplementation {
   // This can't call super(), since that's a factory constructor
   EmptyStyleDeclaration()
     : super._wrap(dom.document.createElement('div').style);
 
   void set cssText(String value) {
     throw new UnsupportedOperationException(
@@ skipping 239 matching lines @@
 
   void set title(String value) {
     throw new UnsupportedOperationException(
       "Title can't be set for document fragments.");
   }
 
   void set webkitdropzone(String value) {
     throw new UnsupportedOperationException(
       "WebKit drop zone can't be set for document fragments.");
   }
-}
+}