Add use counters for SVGCursorElement

Add use counters for SVGCursorElement

This adds one counter for presence of an SVGCursorElement, and one for
if any ComputedStyle references the element (==the SVGCursorElement has
a client.)

BUG=650598
Committed: https://crrev.com/2ee169bbba95ad27368dc8d275b5744335582e47
Cr-Commit-Position: refs/heads/master@{#421630}

[fs, 2016-09-28 13:50:36]

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

[commit-bot: I haz the power, 2016-09-28 13:50:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[fs, 2016-09-28 14:45:19]

fs@opera.com changed reviewers:
+ foolip@chromium.org, isherman@chromium.org, pdr@chromium.org

[foolip, 2016-09-28 14:57:01]

lgtm

https://codereview.chromium.org/2376613004/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/svg/SVGCursorElement.cpp (right):

https://codereview.chromium.org/2376613004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/svg/SVGCursorElement.cpp:50:
UseCounter::count(document(), UseCounter::SVGCursorElementHasClient);
If this is not reach, the whole element has no effect except for sitting in the
DOM?

[fs, 2016-09-28 15:03:02]

https://codereview.chromium.org/2376613004/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/svg/SVGCursorElement.cpp (right):

https://codereview.chromium.org/2376613004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/svg/SVGCursorElement.cpp:50:
UseCounter::count(document(), UseCounter::SVGCursorElementHasClient);
On 2016/09/28 at 14:57:01, foolip wrote:
> If this is not reach, the whole element has no effect except for sitting in
the DOM?

That is correct, yes. Technically we still don't know if it's actually used even
then, but we lose enough information to be unable to determine (count) that.

[commit-bot: I haz the power, 2016-09-28 15:14:54]

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

[commit-bot: I haz the power, 2016-09-28 15:14:55]

Dry run: This issue passed the CQ dry run.

[Ilya Sherman, 2016-09-28 20:04:59]

histograms.xml lgtm

[fs, 2016-09-28 20:29:03]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-09-28 20:29:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-28 20:35:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-28 20:39:47]

Description was changed from

==========
Add use counters for SVGCursorElement

This adds one counter for presence of an SVGCursorElement, and one for
if any ComputedStyle references the element (==the SVGCursorElement has
a client.)

BUG=650598
==========

to

==========
Add use counters for SVGCursorElement

This adds one counter for presence of an SVGCursorElement, and one for
if any ComputedStyle references the element (==the SVGCursorElement has
a client.)

BUG=650598

Committed: https://crrev.com/2ee169bbba95ad27368dc8d275b5744335582e47
Cr-Commit-Position: refs/heads/master@{#421630}
==========

[commit-bot: I haz the power, 2016-09-28 20:39:49]

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