Adding 'supported' check for shadow root.

Adding 'supported' check for shadow root.

This fixes the case for JS, but still throws for Dartium, I'll pass this onto Anton to fix the Dartium case to not throw.


BUG=5537
Committed: https://code.google.com/p/dart/source/detail?r=12986

[Jennifer Messerly, 2012-09-27 21:18:47]

Probably want to change
lib/html/templates/html/dart2js/factoryprovider_ShadowRoot.darttemplate too?

The current create method only checks for the WebKit prefix name:

  static ShadowRoot createShadowRoot(Element host) native '''
      return new WebKitShadowRoot(host);
    ''';


I'm guessing it should be:

new (ShadowRoot || WebKitShadowRoot)(host)

https://codereview.chromium.org/10987080/diff/1/lib/html/templates/html/impl/...
File lib/html/templates/html/impl/impl_ShadowRoot.darttemplate (right):

https://codereview.chromium.org/10987080/diff/1/lib/html/templates/html/impl/...
lib/html/templates/html/impl/impl_ShadowRoot.darttemplate:11: JS('bool',
'!!(window.WebKitShadowRoot || window.ShadowRoot)');
might want to check ShadowRoot first? as it will be the standard name

[Jennifer Messerly, 2012-09-27 21:19:38]

For what it's worth, this lgtm modulo previous comments :)

[blois, 2012-09-27 21:36:31]

https://codereview.chromium.org/10987080/diff/1/lib/html/templates/html/impl/...
File lib/html/templates/html/impl/impl_ShadowRoot.darttemplate (right):

https://codereview.chromium.org/10987080/diff/1/lib/html/templates/html/impl/...
lib/html/templates/html/impl/impl_ShadowRoot.darttemplate:11: JS('bool',
'!!(window.WebKitShadowRoot || window.ShadowRoot)');
On 2012/09/27 21:18:47, John Messerly wrote:
> might want to check ShadowRoot first? as it will be the standard name

Done.

[Jennifer Messerly, 2012-09-27 22:54:23]

lgtm, thank you!

[Anton Muhin, 2012-09-28 10:49:54]

Pete, fill free to reassign the bug to me.

[vsm, 2012-09-28 15:08:46]

lgtm.

Is the convention we want to support elsewhere?