Consolidate PhysicalWeb list launch code

Consolidate PhysicalWeb list launch code

Currently, only the Intent creation code is consolidated to launch an
activity to list Physical Web URLs.  This change also consolidates the
actual call to launch the trusted intent.

BUG=663842
Review-Url: https://codereview.chromium.org/2636243003
Cr-Commit-Position: refs/heads/master@{#444157}
Committed: https://chromium.googlesource.com/chromium/src/+/0df20a42550425c0edc2ca31e76ea5fe1efff67c

[cco3, 2017-01-17 20:16:24]

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

[cco3, 2017-01-17 20:16:46]

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

[cco3, 2017-01-17 20:16:47]

Hi Ted, this is a follow up to https://codereview.chromium.org/2601273003

[mattreynolds, 2017-01-17 20:28:15]

https://codereview.chromium.org/2636243003/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWebDiagnosticsPage.java
(left):

https://codereview.chromium.org/2636243003/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWebDiagnosticsPage.java:66:
PhysicalWebUma.onActivityReferral(ListUrlsActivity.DIAGNOSTICS_REFERER);
What do you think about passing the referer info to showUrlList?

[cco3, 2017-01-17 20:46:42]

I considered it, but I don't know if we should mandate a new referer be
registered if one doesn't already exist.

[Ted C, 2017-01-17 21:02:29]

lgtm

[cco3, 2017-01-17 21:15:22]

The CQ bit was checked by cco3@chromium.org

[commit-bot: I haz the power, 2017-01-17 21:15:44]

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-01-17 21:54:25]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484687722103680, "parent_rev":
"028afa309bb03d8de13dbd76fe5df287fd60d8c9", "commit_rev":
"0df20a42550425c0edc2ca31e76ea5fe1efff67c"}

[commit-bot: I haz the power, 2017-01-17 21:55:09]

Description was changed from

==========
Consolidate PhysicalWeb list launch code

Currently, only the Intent creation code is consolidated to launch an
activity to list Physical Web URLs.  This change also consolidates the
actual call to launch the trusted intent.

BUG=663842
==========

to

==========
Consolidate PhysicalWeb list launch code

Currently, only the Intent creation code is consolidated to launch an
activity to list Physical Web URLs.  This change also consolidates the
actual call to launch the trusted intent.

BUG=663842

Review-Url: https://codereview.chromium.org/2636243003
Cr-Commit-Position: refs/heads/master@{#444157}
Committed:
https://chromium.googlesource.com/chromium/src/+/0df20a42550425c0edc2ca31e76e...
==========

[commit-bot: I haz the power, 2017-01-17 21:55:11]

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