Add UseCounters for Selection methods that take null

Add UseCounters for Selection methods that take null

Making collapse's argument non-nullable appears risky:
https://codereview.chromium.org/1498253002/#msg9

The risk for setBaseAndExtent is entirely unknown.

Measure both cases before proceeding further.

The change to Selection.idl is for clarity only, these are implicitly
nullable because of [LegacyInterfaceTypeChecking], and adding making
them explicitly nullable does not change the generated code.

Committed: https://crrev.com/a1d16968955b0957078049434f840b4b7fc64023
Cr-Commit-Position: refs/heads/master@{#364344}

[philipj_slow, 2015-12-09 11:16:17]

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

[commit-bot: I haz the power, 2015-12-09 11:16:58]

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

[philipj_slow, 2015-12-09 12:54:02]

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

[philipj_slow, 2015-12-09 12:54:02]

PTAL

[haraken, 2015-12-09 13:57:46]

LGTM

[commit-bot: I haz the power, 2015-12-09 14:58:37]

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

[commit-bot: I haz the power, 2015-12-09 14:58:38]

Dry run: This issue passed the CQ dry run.

[Ilya Sherman, 2015-12-09 23:14:55]

histograms.xml lgtm

[philipj_slow, 2015-12-10 10:09:44]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-12-10 10:09:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, isherman@chromium.org
Link to the patchset: https://codereview.chromium.org/1509353004/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2015-12-10 10:10:08]

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

[commit-bot: I haz the power, 2015-12-10 12:46:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-10 12:47:32]

Description was changed from

==========
Add UseCounters for Selection methods that take null

Making collapse's argument non-nullable appears risky:
https://codereview.chromium.org/1498253002/#msg9

The risk for setBaseAndExtent is entirely unknown.

Measure both cases before proceeding further.

The change to Selection.idl is for clarity only, these are implicitly
nullable because of [LegacyInterfaceTypeChecking], and adding making
them explicitly nullable does not change the generated code.
==========

to

==========
Add UseCounters for Selection methods that take null

Making collapse's argument non-nullable appears risky:
https://codereview.chromium.org/1498253002/#msg9

The risk for setBaseAndExtent is entirely unknown.

Measure both cases before proceeding further.

The change to Selection.idl is for clarity only, these are implicitly
nullable because of [LegacyInterfaceTypeChecking], and adding making
them explicitly nullable does not change the generated code.

Committed: https://crrev.com/a1d16968955b0957078049434f840b4b7fc64023
Cr-Commit-Position: refs/heads/master@{#364344}
==========

[commit-bot: I haz the power, 2015-12-10 12:47:33]

Patchset 2 (id:??) landed as
https://crrev.com/a1d16968955b0957078049434f840b4b7fc64023
Cr-Commit-Position: refs/heads/master@{#364344}