[webkit-dev] Enable MICRODATA feature by default

[Ryosuke Niwa]

On Tue, Jul 31, 2012 at 12:55 AM, Arko Saha <nghq36 at motorola.com> wrote:

> For an example :
>
> test(function () {
>     assert_equals( makeEl('div',{itemtype:' '}).itemType[0],
> window.undefined );
> }, 'itemType[index] must be undefined for out-of-range index');
>
> makeEl method creates an element <div> with empty itemtype(0 tokens). Test
> expects itemTpye[0] should return undefined, in our case it returns null.
>
> According to the spec itemtpye attribute is a space-separated list.
> http://www.whatwg.org/specs/web-apps/current-work#attr-itemtype<http://www.whatwg.org/specs/web-apps/current-work/#attr-itemtype>
> We have defined itemtype attribute as :
> readonly attribute [Conditional=MICRODATA] DOMSettableTokenList itemType;
>
> DOMSettableTokenList returns null for out-of-range index.
> So I thought it is specific to DOMSettableTokenList implementation. Or is
> this the bug in our DOMSettableTokenList implementation? There are many
> tests failing because of this.
>

Either that the spec is out of date or that the test is wrong. It's
possible that the spec has recently changed after they had submitted the
test case.

Do you think, we should first resolve all these test issues and then think
> of enabling feature by default? In that case I will raise a new bug and we
> can then discuss the failed tests there.
>

It seems like this particular problem isn't specific to micro data so it's
probably okay. But we should investigate other failures and make sure our
implementation interoperate with Opera and Firefox.

- Ryosuke
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20120731/dda2ac98/attachment.html>