fix findController for non-dart elements

fix findController for non-dart elements

BUG= http://dartbug.com/20931
R=sigmund@google.com

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

[jakemac, 2014-09-18 17:39:43]

jakemac@google.com changed reviewers:
+ sigmund@google.com

[Siggi Cherem (dart-lang), 2014-09-18 18:10:58]

great, I wonder if we should add a simple text (creating manually a couple JS
polymer elements, like we do in the interop test)

https://codereview.chromium.org/582113002/diff/20001/pkg/polymer/lib/src/even...
File pkg/polymer/lib/src/events.dart (right):

https://codereview.chromium.org/582113002/diff/20001/pkg/polymer/lib/src/even...
pkg/polymer/lib/src/events.dart:55: } else if (node.nodeName.contains('-')){
as we discussed, let's switch to:
 - check is Element
 - use e.localName
 - check attributes['is'] as well

[jakemac, 2014-09-18 18:17:08]

Will work on a test now

https://codereview.chromium.org/582113002/diff/20001/pkg/polymer/lib/src/even...
File pkg/polymer/lib/src/events.dart (right):

https://codereview.chromium.org/582113002/diff/20001/pkg/polymer/lib/src/even...
pkg/polymer/lib/src/events.dart:55: } else if (node.nodeName.contains('-')){
On 2014/09/18 18:10:58, Siggi Cherem (dart-lang) wrote:
> as we discussed, let's switch to:
>  - check is Element
>  - use e.localName
>  - check attributes['is'] as well

Done.

[jakemac, 2014-09-18 20:03:34]

Test added, ptal

[Siggi Cherem (dart-lang), 2014-09-18 20:14:42]

lgtm

[jakemac, 2014-09-18 20:31:01]

Committed patchset #4 (id:60001) manually as 40458 (presubmit successful).