Make patch file import declarations apply to the patched library too.

Make patch file import declarations apply to the patched library too.

Add a math library and apply patching to that, instead of to the web library.
We still have a default Math class with the same functionality, which isn't patched.

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

[Johnni Winther, 2012-07-03 11:04:29]

lgtm

https://chromiumcodereview.appspot.com/10694067/diff/1/lib/compiler/implement...
File lib/compiler/implementation/compiler.dart (right):

https://chromiumcodereview.appspot.com/10694067/diff/1/lib/compiler/implement...
lib/compiler/implementation/compiler.dart:274: } catch (var doubleFault) {}
Move the ending } to a new line and insert a comment like 
// Ignore double faults.
in the catch block.

[floitsch, 2012-07-03 12:06:55]

LGTM.

https://chromiumcodereview.appspot.com/10694067/diff/1/lib/compiler/implement...
File lib/compiler/implementation/compiler.dart (right):

https://chromiumcodereview.appspot.com/10694067/diff/1/lib/compiler/implement...
lib/compiler/implementation/compiler.dart:568: if (resolved.scheme == "dart") {
I would prefer this check in 'patchDartLibrary'. This seems to be the wrong
moment to make sure that we don't patch non-core libraries.

[Lasse Reichstein Nielsen, 2012-07-03 12:55:04]

https://chromiumcodereview.appspot.com/10694067/diff/1/lib/compiler/implement...
File lib/compiler/implementation/compiler.dart (right):

https://chromiumcodereview.appspot.com/10694067/diff/1/lib/compiler/implement...
lib/compiler/implementation/compiler.dart:568: if (resolved.scheme == "dart") {
This is not a check to avoid patching non-dart libraries as such. It's a check
to make sure that resolved.path is the name of a dart library, so we can use it
as the second argument to patchDartLibrary. We can't call patchDartLibrary at
all if this isn't the case.

[Lasse Reichstein Nielsen, 2012-07-03 13:01:03]

https://chromiumcodereview.appspot.com/10694067/diff/1/lib/compiler/implement...
File lib/compiler/implementation/compiler.dart (right):

https://chromiumcodereview.appspot.com/10694067/diff/1/lib/compiler/implement...
lib/compiler/implementation/compiler.dart:274: } catch (var doubleFault) {}
On 2012/07/03 11:04:29, Johnni Winther wrote:
> Move the ending } to a new line and insert a comment like 
> // Ignore double faults.
> in the catch block.

Done.