Fix string unifizationf for dartc.

Fix string unifizationf for dartc.

We don't have to expose string.dart anymore so merge it with
string_patch.dart.

BUG=

Committed: https://code.google.com/p/dart/source/detail?r=11381

[Mads Ager (google), 2012-08-27 12:55:03]

Shouldn't you just rename string to string_implementation or something to avoid
the overwriting. This is not a correct use of patching as far as I can tell?

https://chromiumcodereview.appspot.com/10868105/diff/1/corelib/src/string.dart
File corelib/src/string.dart (right):

https://chromiumcodereview.appspot.com/10868105/diff/1/corelib/src/string.dar...
corelib/src/string.dart:11: interface String extends
Move extends to the next line please. One line for extends and one for default?

https://chromiumcodereview.appspot.com/10868105/diff/1/runtime/lib/string_pat...
File runtime/lib/string_patch.dart (right):

https://chromiumcodereview.appspot.com/10868105/diff/1/runtime/lib/string_pat...
runtime/lib/string_patch.dart:23: class StringBase {
If this is in a patch file, doesn't it have to be private? You cannot inject
public classes (should not be able to).

[Anders Johnsen, 2012-08-27 13:00:24]

PTAL

https://chromiumcodereview.appspot.com/10868105/diff/1/corelib/src/string.dart
File corelib/src/string.dart (right):

https://chromiumcodereview.appspot.com/10868105/diff/1/corelib/src/string.dar...
corelib/src/string.dart:11: interface String extends
On 2012/08/27 12:55:03, Mads Ager wrote:
> Move extends to the next line please. One line for extends and one for
default?

Yes ofc, done.

[Mads Ager (google), 2012-08-27 13:14:30]

lgtm

[Mads Ager (google), 2012-08-27 13:14:30]

lgtm