Measure SVGPathElement.getPathSegAtLength usage

Measure SVGPathElement.getPathSegAtLength usage

This interface removed from Spec.
Measuring this usage will be useful to remove this.

BUG=669498
Review-Url: https://codereview.chromium.org/2814723002
Cr-Commit-Position: refs/heads/master@{#463948}
Committed: https://chromium.googlesource.com/chromium/src/+/89b08748a1f0bef747dd1b15f3e80ad9dc70c7b9

[Shanmuga Pandi, 2017-04-11 14:07:42]

Description was changed from

==========
Measure SVGPathElement.getPathSegAtLength usage

This interface removed from Spec.
Measuring this usage will be useful to remove this.

BUG=669498
==========

to

==========
Measure SVGPathElement.getPathSegAtLength usage

This interface removed from Spec.
Measuring this usage will be useful to remove this.

BUG=669498
==========

[Shanmuga Pandi, 2017-04-11 14:07:43]

shanmuga.m@samsung.com changed reviewers:
+ foolip@chromium.org, fs@opera.com, isherman@chromium.org

[Shanmuga Pandi, 2017-04-11 14:07:59]

PTAL!!

[foolip, 2017-04-11 14:20:01]

The CQ bit was checked by foolip@chromium.org

[foolip, 2017-04-11 14:20:01]

lgtm

[commit-bot: I haz the power, 2017-04-11 14:20:27]

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

[foolip, 2017-04-11 14:22:58]

FWIW, there are only 15 hits for "getPathSegAtLength" in httparchive, making it
quite likely that usage will be very low.

Nevertheless, there's no particular hurry here, so waiting for use counter data
is fine. If you'd like, however, I think looking at the httparchive usage and
deprecating with a tentative removal date would be fine. Then we just make sure
to check the usage from the stable channel before actually going through with
the removal.

[commit-bot: I haz the power, 2017-04-11 14:30:05]

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

[commit-bot: I haz the power, 2017-04-11 14:30:06]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Ilya Sherman, 2017-04-11 21:37:22]

histograms.xml lgtm

[foolip, 2017-04-12 04:39:33]

The CQ bit was checked by foolip@chromium.org

[commit-bot: I haz the power, 2017-04-12 04:39:54]

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, 2017-04-12 07:49:47]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491971973125720, "parent_rev":
"9871ced334b179dbe05c98c9e99bca5114a3bc5e", "commit_rev":
"89b08748a1f0bef747dd1b15f3e80ad9dc70c7b9"}

[commit-bot: I haz the power, 2017-04-12 07:50:30]

Description was changed from

==========
Measure SVGPathElement.getPathSegAtLength usage

This interface removed from Spec.
Measuring this usage will be useful to remove this.

BUG=669498
==========

to

==========
Measure SVGPathElement.getPathSegAtLength usage

This interface removed from Spec.
Measuring this usage will be useful to remove this.

BUG=669498

Review-Url: https://codereview.chromium.org/2814723002
Cr-Commit-Position: refs/heads/master@{#463948}
Committed:
https://chromium.googlesource.com/chromium/src/+/89b08748a1f0bef747dd1b15f3e8...
==========

[commit-bot: I haz the power, 2017-04-12 07:50:31]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/89b08748a1f0bef747dd1b15f3e8...

[Shanmuga Pandi, 2017-04-13 10:31:09]

On 2017/04/11 14:22:58, foolip_UTC7 wrote:
> FWIW, there are only 15 hits for "getPathSegAtLength" in httparchive, making
it
> quite likely that usage will be very low.
> 
> Nevertheless, there's no particular hurry here, so waiting for use counter
data
> is fine. If you'd like, however, I think looking at the httparchive usage and
> deprecating with a tentative removal date would be fine. Then we just make
sure
> to check the usage from the stable channel before actually going through with
> the removal.

Thanks foolip.
Shall I post Intent to Deprecate to blink-dev without use counter ?

[foolip, 2017-04-13 10:41:16]

On 2017/04/13 10:31:09, Shanmuga Pandi wrote:
> On 2017/04/11 14:22:58, foolip_UTC7 wrote:
> > FWIW, there are only 15 hits for "getPathSegAtLength" in httparchive, making
> it
> > quite likely that usage will be very low.
> > 
> > Nevertheless, there's no particular hurry here, so waiting for use counter
> data
> > is fine. If you'd like, however, I think looking at the httparchive usage
and
> > deprecating with a tentative removal date would be fine. Then we just make
> sure
> > to check the usage from the stable channel before actually going through
with
> > the removal.
> 
> Thanks foolip.
> Shall I post Intent to Deprecate to blink-dev without use counter ?

Yes, in this case I think it's OK to proceed with the deprecation and decide a
tentative milestone for removal before we have the use counter data. So that we
have the data before the removal hits stable, I think at least two milestones of
deprecation are necessary. So if you suggest to deprecate in M60, remove in M62.