Revert "dart2js implementation of StringBuffer.writeAll that optimizes better."

tests/corelib/string_buffer_test.dart

Index: tests/corelib/string_buffer_test.dart
diff --git a/tests/corelib/string_buffer_test.dart b/tests/corelib/string_buffer_test.dart
index 8a24b35a306d702236de56df9f7a21e103bc1e26..2c556f79109515b4842e543158575ec7a816db3e 100644
--- a/tests/corelib/string_buffer_test.dart
+++ b/tests/corelib/string_buffer_test.dart
@@ -96,49 +96,6 @@ void testWriteAll() {
 
   bf.writeAll(["", "", ""]);
   Expect.equals("foobarabc", bf.toString());
-
-  bf.writeAll(["", "", ""], "");
-  Expect.equals("foobarabc", bf.toString());
-
-  StringBuffer bf2 = new StringBuffer("");
-  bf2.writeAll([], "s");
-  Expect.equals("", bf2.toString());
-
-  StringBuffer bf3 = new StringBuffer("");
-  bf3.writeAll(["a"], "s");
-  Expect.equals("a", bf3.toString());
-
-  StringBuffer bf4 = new StringBuffer("");
-  bf4.writeAll(["a", "b"], "s");
-  Expect.equals("asb", bf4.toString());
-}
-
-void testWriteAll2() {
-  // Passing `null` for separator is an error that is checked when the iterable
-  // is not empty.  This is not specified in the documentation but we want
-  // implementations to be consistent.
-  StringBuffer bf1 = new StringBuffer("");
-  bf1.writeAll([], null);
-  Expect.equals("", bf1.toString());
-
-  StringBuffer bf2 = new StringBuffer("");
-  Expect.throws(() { bf2.writeAll([1], null); });
-}
-
-void testWriteln() {
-  StringBuffer bf1 = new StringBuffer("");
-  bf1.writeln("Hello");
-  Expect.equals("Hello\n", bf1.toString());
-
-  StringBuffer bf2 = new StringBuffer("");
-  bf2.writeln();
-  Expect.equals("\n", bf2.toString());
-
-  StringBuffer bf3 = new StringBuffer("");
-  bf3.writeln("\n");
-  bf3.writeln(null);
-  bf3.writeln(1);
-  Expect.equals("\n\nnull\n1\n", bf3.toString());
 }
 
 void testClear() {
@@ -229,8 +186,6 @@ void main() {
   testWrite();
   testWriteCharCode();
   testWriteAll();
-  testWriteAll2();
-  testWriteln();
   testClear();
   testChaining();
 }