Measure the many aspects of HTMLAllCollection

Measure the many aspects of HTMLAllCollection

Multiple spec changes to HTMLAllCollection are under discussion:
https://github.com/whatwg/html/issues/775
https://github.com/whatwg/html/pull/780

To help inform the discussion and estimate risk, measure some of the
aspects that are currently not per spec, as well as a few extra bits for
comparison purposes.

BUG=591605
Committed: https://crrev.com/69256151c9e821963929cf29a778b5df66027023
Cr-Commit-Position: refs/heads/master@{#379484}

[philipj_slow, 2016-03-02 18:01:26]

The CQ bit was checked by philipj@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-02 18:02:20]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1756963002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1756963002/1

[commit-bot: I haz the power, 2016-03-02 20:32:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-02 20:32:59]

Dry run: This issue passed the CQ dry run.

[philipj_slow, 2016-03-03 18:12:00]

Description was changed from

==========
Count some things about HTMLAllCollection

BUG=
==========

to

==========
Measure 0- and 2-argument uses of HTMLAllCollection

The 2-argument form is not in the spec, and the 0-argument form does
nothing and is very non-interoperable, so I'd like to know if it's
really needed for compat. See https://github.com/whatwg/html/issues/775

BUG=591605
==========

[philipj_slow, 2016-03-03 18:12:15]

philipj@opera.com changed reviewers:
+ domenic@chromium.org, haraken@chromium.org

[philipj_slow, 2016-03-03 18:12:15]

PTAL

[domenic, 2016-03-03 18:14:42]

On 2016/03/03 at 18:12:15, philipj wrote:
> PTAL

The 0 argument form is actually very interoperable for item(). Firefox is the
only one that does not implement it. legaycaller is slightly less as Edge also
does not, but given that we're aligning legacycaller with item(), I don't think
anyone should bother removing it.

[philipj_slow, 2016-03-03 18:40:26]

On 2016/03/03 18:14:42, domenic wrote:
> On 2016/03/03 at 18:12:15, philipj wrote:
> > PTAL
> 
> The 0 argument form is actually very interoperable for item(). Firefox is the
> only one that does not implement it. legaycaller is slightly less as Edge also
> does not, but given that we're aligning legacycaller with item(), I don't
think
> anyone should bother removing it.

It's interoperable only in the sense that it doesn't throw, but see
https://github.com/whatwg/html/issues/775#issuecomment-191632695 for what the
return values are. Neither document.all() or document.all.item() does something
useful in any browser, and it's different kinds of useless.

I've made lots of arguments non-optional that were previously optional, in cases
that seemed riskier than this. And document.all.namedItem was changed, although
not by me. I think it's worth *something* to throw for nonsensical uses of an
API, although if usage turns out to be even 0.0001% I'll drop this idea and
carry on.

[philipj_slow, 2016-03-05 17:47:36]

Description was changed from

==========
Measure 0- and 2-argument uses of HTMLAllCollection

The 2-argument form is not in the spec, and the 0-argument form does
nothing and is very non-interoperable, so I'd like to know if it's
really needed for compat. See https://github.com/whatwg/html/issues/775

BUG=591605
==========

to

==========
Measure the many aspects of HTMLAllCollection

Multiple spec changes to HTMLAllCollection are under discussion:
https://github.com/whatwg/html/issues/775
https://github.com/whatwg/html/pull/780

To help inform the discussion and estimate risk, measure some of the
aspects that are currently not per spec, as well as a few extra bits for
comparison purposes.

BUG=591605
==========

[philipj_slow, 2016-03-05 17:47:50]

Description was changed from

==========
Measure the many aspects of HTMLAllCollection

Multiple spec changes to HTMLAllCollection are under discussion:
https://github.com/whatwg/html/issues/775
https://github.com/whatwg/html/pull/780

To help inform the discussion and estimate risk, measure some of the
aspects that are currently not per spec, as well as a few extra bits for
comparison purposes.

BUG=591605
==========

to

==========
Measure the many aspects of HTMLAllCollection

Multiple spec changes to HTMLAllCollection are under discussion:
https://github.com/whatwg/html/issues/775
https://github.com/whatwg/html/pull/780

To help inform the discussion and estimate risk, measure some of the
aspects that are currently not per spec, as well as a few extra bits for
comparison purposes.

BUG=591605
==========

[philipj_slow, 2016-03-05 17:49:43]

philipj@opera.com changed reviewers:
+ isherman@chromium.org

[philipj_slow, 2016-03-05 17:49:43]

I've expanded the scope of this CL to measure everything that could potentially
be useful in deciding what to do with the many behaviors that are under
discussion. PTAL.

[Ilya Sherman, 2016-03-06 03:42:54]

histograms.xml lgtm

[haraken, 2016-03-06 03:51:44]

bindings LGTM

[philipj_slow, 2016-03-06 05:02:01]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2016-03-06 05:02:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1756963002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1756963002/40001

[commit-bot: I haz the power, 2016-03-06 05:08:22]

Description was changed from

==========
Measure the many aspects of HTMLAllCollection

Multiple spec changes to HTMLAllCollection are under discussion:
https://github.com/whatwg/html/issues/775
https://github.com/whatwg/html/pull/780

To help inform the discussion and estimate risk, measure some of the
aspects that are currently not per spec, as well as a few extra bits for
comparison purposes.

BUG=591605
==========

to

==========
Measure the many aspects of HTMLAllCollection

Multiple spec changes to HTMLAllCollection are under discussion:
https://github.com/whatwg/html/issues/775
https://github.com/whatwg/html/pull/780

To help inform the discussion and estimate risk, measure some of the
aspects that are currently not per spec, as well as a few extra bits for
comparison purposes.

BUG=591605
==========

[commit-bot: I haz the power, 2016-03-06 05:08:23]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-06 05:09:47]

Description was changed from

==========
Measure the many aspects of HTMLAllCollection

Multiple spec changes to HTMLAllCollection are under discussion:
https://github.com/whatwg/html/issues/775
https://github.com/whatwg/html/pull/780

To help inform the discussion and estimate risk, measure some of the
aspects that are currently not per spec, as well as a few extra bits for
comparison purposes.

BUG=591605
==========

to

==========
Measure the many aspects of HTMLAllCollection

Multiple spec changes to HTMLAllCollection are under discussion:
https://github.com/whatwg/html/issues/775
https://github.com/whatwg/html/pull/780

To help inform the discussion and estimate risk, measure some of the
aspects that are currently not per spec, as well as a few extra bits for
comparison purposes.

BUG=591605

Committed: https://crrev.com/69256151c9e821963929cf29a778b5df66027023
Cr-Commit-Position: refs/heads/master@{#379484}
==========

[commit-bot: I haz the power, 2016-03-06 05:09:48]

Patchset 3 (id:??) landed as
https://crrev.com/69256151c9e821963929cf29a778b5df66027023
Cr-Commit-Position: refs/heads/master@{#379484}