Stop importing js_helper.dart explicitly in the core library.

Remove direct imports of js_helper.dart.

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

[Lasse Reichstein Nielsen, 2012-09-24 09:32:05]

Similar to our previously discussed change.

Can't be committed until the mirror system can handle Dynamic as a type that
don't belong to a library in the system (because this change makes "js_helper" a
library that isn't in compiler.libraries, and Dynamic is declared in there).

[ahe, 2012-09-24 10:54:59]

I'd like to split the compiler changes from the library changes. Small changes
makes it possible for me to review in finite time :-)

[Lasse Reichstein Nielsen, 2012-09-24 11:47:16]

PTAL

[ahe, 2012-09-24 12:57:33]

On 2012/09/24 11:47:16, Lasse Reichstein Nielsen wrote:
> PTAL

Where are the compiler changes?

[ahe, 2012-09-24 12:57:57]

On 2012/09/24 12:57:33, ahe wrote:
> On 2012/09/24 11:47:16, Lasse Reichstein Nielsen wrote:
> > PTAL
> 
> Where are the compiler changes?

Ignore. I was confused.

[Lasse Reichstein Nielsen, 2012-09-24 13:52:03]

https://codereview.chromium.org/10967068/diff/4001/lib/compiler/implementatio...
File lib/compiler/implementation/compiler.dart (right):

https://codereview.chromium.org/10967068/diff/4001/lib/compiler/implementatio...
lib/compiler/implementation/compiler.dart:385: identicalFunction =
coreLibrary.find(const SourceString('identical'));
These two look like they would belong in (a generalized)
initializeSpecialClasses instead of here. Should I move them?

[Lasse Reichstein Nielsen, 2012-09-26 14:36:27]

PTAL :)

[Lasse Reichstein Nielsen, 2012-10-05 08:09:32]

Ping

[ahe, 2012-10-05 08:16:17]

LGTM!