Remove Comparable interface from UrlInfo

Remove Comparable interface from UrlInfo

There are different ways we compare the UrlInfo class.  There is no
reason for it to be comparable instead of just using a custom
Comparator each time as needed (as we already do).

BUG=628711
Committed: https://crrev.com/bfb5deec916198eed2b2edf2621fb7142116faa8
Cr-Commit-Position: refs/heads/master@{#405875}

[cco3, 2016-07-15 18:57:50]

cco3@chromium.org changed reviewers:
+ dfalcantara@chromium.org, mattreynolds@chromium.org

[gone, 2016-07-15 21:13:47]

This addresses a couple of them, but there's still one Lint warning in
UrlManager:

org/chromium/chrome/browser/physicalweb/UrlInfo.class Call requires API level 19
(current min is 16): `java.lang.Boolean#compare`: NewApi [warning]

org/chromium/chrome/browser/physicalweb/UrlInfo.class Call requires API level 19
(current min is 16): `java.lang.Long#compare`: NewApi [warning]

org/chromium/chrome/browser/physicalweb/UrlManager$1.class Call requires API
level 19 (current min is 16): `java.lang.Long#compare`: NewApi [warning]

[gone, 2016-07-15 21:14:25]

(lgtm for this one though)

[cco3, 2016-07-15 21:24:19]

On 2016/07/15 21:13:47, dfalcantara wrote:
> This addresses a couple of them, but there's still one Lint warning in
> UrlManager:
> 
> org/chromium/chrome/browser/physicalweb/UrlInfo.class Call requires API level
19
> (current min is 16): `java.lang.Boolean#compare`: NewApi [warning]
> 
> org/chromium/chrome/browser/physicalweb/UrlInfo.class Call requires API level
19
> (current min is 16): `java.lang.Long#compare`: NewApi [warning]
> 
> org/chromium/chrome/browser/physicalweb/UrlManager$1.class Call requires API
> level 19 (current min is 16): `java.lang.Long#compare`: NewApi [warning]

Right, understood.

[cco3, 2016-07-15 21:24:22]

The CQ bit was checked by cco3@chromium.org

[commit-bot: I haz the power, 2016-07-15 21:24:59]

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

[mattreynolds, 2016-07-15 21:25:10]

lgtm

[commit-bot: I haz the power, 2016-07-15 22:09:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-15 22:10:35]

Description was changed from

==========
Remove Comparable interface from UrlInfo

There are different ways we compare the UrlInfo class.  There is no
reason for it to be comparable instead of just using a custom
Comparator each time as needed (as we already do).

BUG=628711
==========

to

==========
Remove Comparable interface from UrlInfo

There are different ways we compare the UrlInfo class.  There is no
reason for it to be comparable instead of just using a custom
Comparator each time as needed (as we already do).

BUG=628711

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

[commit-bot: I haz the power, 2016-07-15 22:10:36]

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