Second round of cleanups for new optional parameter semantics.

tests/compiler/dart2js/patch_test.dart

 // Copyright (c) 2012, 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.
 
 #import("../../../lib/compiler/implementation/leg.dart");
 #import("../../../lib/compiler/implementation/elements/elements.dart");
 #import("../../../lib/compiler/implementation/tree/tree.dart");
 #import("../../../lib/compiler/implementation/util/util.dart");
 #import("mock_compiler.dart");
 #import("parser_helper.dart");
@@ skipping 26 matching lines @@
     Expect.isNotNull(node.body);
     // If the element has no body it is a Block with identical begin and end
     // tokens (the semicolon).
     Expect.isTrue(node.body is Block);
     Expect.identical(node.body.getBeginToken(), node.body.getEndToken());
 }
 
 Element ensure(compiler,
                String name,
                Element lookup(name),
-               [bool isPatched = false,
-                bool isPatch = false,
-                bool isMethod = true,
-                bool isGetter = false,
-                bool isFound = true]) {
+               {bool isPatched: false,
+                bool isPatch: false,
+                bool isMethod: true,
+                bool isGetter: false,
+                bool isFound: true}) {
   var element = lookup(buildSourceString(name));
   if (!isFound) {
     Expect.isNull(element);
     return element;
   }
   Expect.isNotNull(element);
   if (isGetter) {
     Expect.isTrue(element is AbstractFieldElement);
     Expect.isNotNull(element.getter);
     element = element.getter;
@@ skipping 299 matching lines @@
 
 main() {
   testPatchFunction();
   testPatchMember();
   testPatchGetter();
   testRegularMember();
   testGhostMember();
   testInjectFunction();
   testPatchSignatureCheck();
 }