Fix DOM constructors in inline JS

Fix DOM constructors in inline JS

R=jacobr@google.com

Committed: https://github.com/dart-lang/sdk/commit/50ccf6d9b5d562619752860f92960ff3ce7fcbe1

[vsm, 2017-01-27 13:36:00]

vsm@google.com changed reviewers:
+ jacobr@google.com

[Jacob, 2017-01-27 16:10:31]

lgtm

https://codereview.chromium.org/2659063002/diff/1/pkg/dev_compiler/lib/src/co...
File pkg/dev_compiler/lib/src/compiler/code_generator.dart (right):

https://codereview.chromium.org/2659063002/diff/1/pkg/dev_compiler/lib/src/co...
pkg/dev_compiler/lib/src/compiler/code_generator.dart:3733: // in dart:html
itself.
Alternately, only apply this rename if there is a top level Dart name in this
library that matches the specified JS name.

[vsm, 2017-01-27 16:47:03]

On 2017/01/27 16:10:31, Jacob wrote:
> lgtm
> 
>
https://codereview.chromium.org/2659063002/diff/1/pkg/dev_compiler/lib/src/co...
> File pkg/dev_compiler/lib/src/compiler/code_generator.dart (right):
> 
>
https://codereview.chromium.org/2659063002/diff/1/pkg/dev_compiler/lib/src/co...
> pkg/dev_compiler/lib/src/compiler/code_generator.dart:3733: // in dart:html
> itself.
> Alternately, only apply this rename if there is a top level Dart name in this
> library that matches the specified JS name.

PTAL - restricted this to matching the containing class, where the containing
class is an extension.

[Jacob, 2017-01-27 17:34:32]

lgtm
has to be a record for number of real dart:html fixes in one cl.

[vsm, 2017-01-27 17:47:22]

Description was changed from

==========
Fix DOM constructors in inline JS
==========

to

==========
Fix DOM constructors in inline JS

R=jacobr@google.com

Committed:
https://github.com/dart-lang/sdk/commit/50ccf6d9b5d562619752860f92960ff3ce7fcbe1
==========

[vsm, 2017-01-27 17:47:24]

Committed patchset #2 (id:20001) manually as
50ccf6d9b5d562619752860f92960ff3ce7fcbe1 (presubmit successful).