Add a use counter for Attr.prototype.cloneNode()

Add a use counter for Attr.prototype.cloneNode()

This is being measured as requested in a spec discussion:
https://github.com/whatwg/dom/issues/102#issuecomment-158833267

To avoid making a observable change to Attr.prototype, measure this
internally. This risks including internal uses, but there appear to be
none. All calls were checked and they seem to be for either parent or
child nodes of some kind, and Attr cannot be a parent or child.

Also, a deprecation message was added to see if it would be triggered
unexpectedly on any tests, but it was only emitted for tests that
actually clone attributes, as expected.

Drive-by: Drop an unused ShadowRoot::cloneNode() that was missed in
https://codereview.chromium.org/1482433003

BUG=305105
Committed: https://crrev.com/fff1c216222319d3f6030744810e236df195fb1a
Cr-Commit-Position: refs/heads/master@{#361932}

[philipj_slow, 2015-11-26 14:18:33]

Description was changed from

==========
Add a use counter for Attr.prototype.cloneNode()

BUG=305105
==========

to

==========
Add a use counter for Attr.prototype.cloneNode()

This is being measured as requested in a spec discussion:
https://github.com/whatwg/dom/issues/102#issuecomment-158833267

To avoid making a observable change to Attr.prototype, measure this
internally. This risks including internal uses, but there appear to be
none. All calls were checked and they seem to be for either parent or
child nodes of some kind, and Attr cannot be a parent or child.

Also, a deprecation message was added to see if it would be triggered
unexpectedly on any tests, but it was only emitted for tests that
actually clone attributes, as expected.

BUG=305105
==========

[philipj_slow, 2015-11-26 14:19:50]

Description was changed from

==========
Add a use counter for Attr.prototype.cloneNode()

This is being measured as requested in a spec discussion:
https://github.com/whatwg/dom/issues/102#issuecomment-158833267

To avoid making a observable change to Attr.prototype, measure this
internally. This risks including internal uses, but there appear to be
none. All calls were checked and they seem to be for either parent or
child nodes of some kind, and Attr cannot be a parent or child.

Also, a deprecation message was added to see if it would be triggered
unexpectedly on any tests, but it was only emitted for tests that
actually clone attributes, as expected.

BUG=305105
==========

to

==========
Add a use counter for Attr.prototype.cloneNode()

This is being measured as requested in a spec discussion:
https://github.com/whatwg/dom/issues/102#issuecomment-158833267

To avoid making a observable change to Attr.prototype, measure this
internally. This risks including internal uses, but there appear to be
none. All calls were checked and they seem to be for either parent or
child nodes of some kind, and Attr cannot be a parent or child.

Also, a deprecation message was added to see if it would be triggered
unexpectedly on any tests, but it was only emitted for tests that
actually clone attributes, as expected.

Drive-by: Drop an unused ShadowRoot::cloneNode() that was missed in
https://codereview.chromium.org/1482433003

BUG=305105
==========

[philipj_slow, 2015-11-26 14:20:31]

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

[philipj_slow, 2015-11-26 14:20:31]

PTAL. I know this will conflict so I haven't started the bots, see PS1 for the
bot run mentioned in the description.

[haraken, 2015-11-26 15:12:18]

LGTM

[Ilya Sherman, 2015-11-26 20:52:21]

histograms.xml lgtm

[philipj_slow, 2015-11-26 21:46:10]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-11-26 21:46:14]

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

[commit-bot: I haz the power, 2015-11-26 23:33:51]

Description was changed from

==========
Add a use counter for Attr.prototype.cloneNode()

This is being measured as requested in a spec discussion:
https://github.com/whatwg/dom/issues/102#issuecomment-158833267

To avoid making a observable change to Attr.prototype, measure this
internally. This risks including internal uses, but there appear to be
none. All calls were checked and they seem to be for either parent or
child nodes of some kind, and Attr cannot be a parent or child.

Also, a deprecation message was added to see if it would be triggered
unexpectedly on any tests, but it was only emitted for tests that
actually clone attributes, as expected.

Drive-by: Drop an unused ShadowRoot::cloneNode() that was missed in
https://codereview.chromium.org/1482433003

BUG=305105
==========

to

==========
Add a use counter for Attr.prototype.cloneNode()

This is being measured as requested in a spec discussion:
https://github.com/whatwg/dom/issues/102#issuecomment-158833267

To avoid making a observable change to Attr.prototype, measure this
internally. This risks including internal uses, but there appear to be
none. All calls were checked and they seem to be for either parent or
child nodes of some kind, and Attr cannot be a parent or child.

Also, a deprecation message was added to see if it would be triggered
unexpectedly on any tests, but it was only emitted for tests that
actually clone attributes, as expected.

Drive-by: Drop an unused ShadowRoot::cloneNode() that was missed in
https://codereview.chromium.org/1482433003

BUG=305105
==========

[commit-bot: I haz the power, 2015-11-26 23:33:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-26 23:34:46]

Description was changed from

==========
Add a use counter for Attr.prototype.cloneNode()

This is being measured as requested in a spec discussion:
https://github.com/whatwg/dom/issues/102#issuecomment-158833267

To avoid making a observable change to Attr.prototype, measure this
internally. This risks including internal uses, but there appear to be
none. All calls were checked and they seem to be for either parent or
child nodes of some kind, and Attr cannot be a parent or child.

Also, a deprecation message was added to see if it would be triggered
unexpectedly on any tests, but it was only emitted for tests that
actually clone attributes, as expected.

Drive-by: Drop an unused ShadowRoot::cloneNode() that was missed in
https://codereview.chromium.org/1482433003

BUG=305105
==========

to

==========
Add a use counter for Attr.prototype.cloneNode()

This is being measured as requested in a spec discussion:
https://github.com/whatwg/dom/issues/102#issuecomment-158833267

To avoid making a observable change to Attr.prototype, measure this
internally. This risks including internal uses, but there appear to be
none. All calls were checked and they seem to be for either parent or
child nodes of some kind, and Attr cannot be a parent or child.

Also, a deprecation message was added to see if it would be triggered
unexpectedly on any tests, but it was only emitted for tests that
actually clone attributes, as expected.

Drive-by: Drop an unused ShadowRoot::cloneNode() that was missed in
https://codereview.chromium.org/1482433003

BUG=305105

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

[commit-bot: I haz the power, 2015-11-26 23:34:46]

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