Fixes for the upcoming sdk changes (bleeding_edge 16052)

Fixes for the upcoming sdk changes (bleeding_edge 16052)

Committed: https://github.com/dart-lang/web-ui/commit/13fcc2b

[Jennifer Messerly, 2012-12-12 19:47:18]

https://codereview.chromium.org/11416379/diff/1/lib/src/emitters.dart
File lib/src/emitters.dart (right):

https://codereview.chromium.org/11416379/diff/1/lib/src/emitters.dart#newcode519
lib/src/emitters.dart:519: 'autogenerated.insertAllBefore($id.parentNode,
$id.nextNode,');
+2 spaces?

https://codereview.chromium.org/11416379/diff/1/lib/src/emitters.dart#newcode527
lib/src/emitters.dart:527: printer.add('$id.parentNode.insertBefore($exp,
$id.nextNode);');
Can we keep using using:

http://api.dartlang.org/docs/bleeding_edge/dart_html/Node.html#parent

?

(I don't think parent would ever be DocumentFragment or CharacterData ... )

[Siggi Cherem (dart-lang), 2012-12-12 19:57:42]

also updated the the version number...

https://chromiumcodereview.appspot.com/11416379/diff/1/lib/src/emitters.dart
File lib/src/emitters.dart (right):

https://chromiumcodereview.appspot.com/11416379/diff/1/lib/src/emitters.dart#...
lib/src/emitters.dart:519: 'autogenerated.insertAllBefore($id.parentNode,
$id.nextNode,');
On 2012/12/12 19:47:18, John Messerly wrote:
> +2 spaces?

Done.

https://chromiumcodereview.appspot.com/11416379/diff/1/lib/src/emitters.dart#...
lib/src/emitters.dart:527: printer.add('$id.parentNode.insertBefore($exp,
$id.nextNode);');
On 2012/12/12 19:47:18, John Messerly wrote:
> Can we keep using using:
> 
> http://api.dartlang.org/docs/bleeding_edge/dart_html/Node.html#parent
> 
> ?
> 
> (I don't think parent would ever be DocumentFragment or CharacterData ... )

I tried, but I was hitting bugs with ShadowRoot being a document fragment.

[Jennifer Messerly, 2012-12-12 20:33:02]

lgtm!