Combine Physical Web URLs into groups

Combine Physical Web URLs into groups

A group id is an identifier that associates URLs into buckets.  (i.e.,
URLs from the same domain with the same title might have the same group
id).

This change displays groups as a single entry in our display list, and
then chooses the closest URL when a user taps the entry.

BUG=633715
Committed: https://crrev.com/5ba64b349e320be7a35c42dcdfbdeaea9ae412b3
Cr-Commit-Position: refs/heads/master@{#411724}

[cco3, 2016-08-08 23:15:04]

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

[cco3, 2016-08-08 23:15:05]

Hi Matt, PTAL

[mattreynolds, 2016-08-09 18:41:49]

lgtm, thanks for adding the tests

[cco3, 2016-08-09 18:44:27]

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

[gone, 2016-08-10 00:29:18]

lgtm

https://chromiumcodereview.appspot.com/2224223002/diff/1/chrome/android/java/...
File
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivity.java
(right):

https://chromiumcodereview.appspot.com/2224223002/diff/1/chrome/android/java/...
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivity.java:78:
private List<PwsResult> mPwsResults;
private final List<PwsResult> mPwsResults = new ArrayList<>()?

[cco3, 2016-08-12 18:22:10]

https://codereview.chromium.org/2224223002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivity.java
(right):

https://codereview.chromium.org/2224223002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivity.java:78:
private List<PwsResult> mPwsResults;
On 2016/08/10 00:29:18, dfalcantara wrote:
> private final List<PwsResult> mPwsResults = new ArrayList<>()?

Done.

[cco3, 2016-08-12 18:22:14]

The CQ bit was checked by cco3@chromium.org

[cco3, 2016-08-12 18:22:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
dfalcantara@chromium.org, mattreynolds@chromium.org
Link to the patchset: https://codereview.chromium.org/2224223002/#ps20001
(title: "Make mPwsResults final")

[commit-bot: I haz the power, 2016-08-12 18:22:48]

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-08-12 19:05:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-12 19:07:50]

Description was changed from

==========
Combine Physical Web URLs into groups

A group id is an identifier that associates URLs into buckets.  (i.e.,
URLs from the same domain with the same title might have the same group
id).

This change displays groups as a single entry in our display list, and
then chooses the closest URL when a user taps the entry.

BUG=633715
==========

to

==========
Combine Physical Web URLs into groups

A group id is an identifier that associates URLs into buckets.  (i.e.,
URLs from the same domain with the same title might have the same group
id).

This change displays groups as a single entry in our display list, and
then chooses the closest URL when a user taps the entry.

BUG=633715

Committed: https://crrev.com/5ba64b349e320be7a35c42dcdfbdeaea9ae412b3
Cr-Commit-Position: refs/heads/master@{#411724}
==========

[commit-bot: I haz the power, 2016-08-12 19:07:51]

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