arc: Remove IntentHelperInstance::HandleUrlListDeprecated.

arc: Remove IntentHelperInstance::HandleUrlListDeprecated.

Remove IntentHelperInstance's HandleUrlListDeprecated
method. Chrome stopped calling it in
https://codereview.chromium.org/2392253003/.

BUG=647802
Committed: https://crrev.com/ca145c8a83b4c38726d09412e9a716764fadb980
Cr-Commit-Position: refs/heads/master@{#423798}

[Daniel Erat, 2016-10-07 00:57:44]

derat@chromium.org changed reviewers:
+ dcheng@chromium.org, kinaba@chromium.org, yusukes@chromium.org

[Daniel Erat, 2016-10-07 00:57:45]

is there a canonical place to check to determine when it's safe to remove a
method that's no longer being called? it's been about a day since i made chrome
stop calling this.

[Daniel Erat, 2016-10-07 00:57:50]

The CQ bit was checked by derat@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-07 00:58:07]

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

[kinaba, 2016-10-07 01:08:09]

lgtm

On 2016/10/07 00:57:45, Daniel Erat wrote:
> is there a canonical place to check to determine when it's safe to remove a
> method that's no longer being called? it's been about a day since i made
chrome
> stop calling this.

For this particular case it should be safe anyway.
Already at the 3 months ago when the "deprecated" name introduced,
the method was not called if the callee version is new enough.

In general, we at least wait for Chrome roll to the Chrome OS tree
https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+log...
catches relevant changes.

[commit-bot: I haz the power, 2016-10-07 01:24:29]

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

[commit-bot: I haz the power, 2016-10-07 01:24:30]

Dry run: This issue passed the CQ dry run.

[dcheng, 2016-10-07 03:29:58]

rs lgtm for mojom (yay for code deletion)

[Yusuke Sato, 2016-10-07 03:35:39]

lgtm

[Daniel Erat, 2016-10-07 05:03:41]

The CQ bit was checked by derat@chromium.org

[commit-bot: I haz the power, 2016-10-07 05:04:01]

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-10-07 05:09:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-07 05:10:45]

Description was changed from

==========
arc: Remove IntentHelperInstance::HandleUrlListDeprecated.

Remove IntentHelperInstance's HandleUrlListDeprecated
method. Chrome stopped calling it in
https://codereview.chromium.org/2392253003/.

BUG=647802
==========

to

==========
arc: Remove IntentHelperInstance::HandleUrlListDeprecated.

Remove IntentHelperInstance's HandleUrlListDeprecated
method. Chrome stopped calling it in
https://codereview.chromium.org/2392253003/.

BUG=647802

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

[commit-bot: I haz the power, 2016-10-07 05:10:47]

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