[Android History] Add support for opening items

[Android History] Add support for opening items

BUG=654071
Committed: https://crrev.com/3ba7a94420937d5ee40e62afbb0d730a7f751f89
Cr-Commit-Position: refs/heads/master@{#437140}

[Theresa, 2016-12-06 21:56:00]

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

[commit-bot: I haz the power, 2016-12-06 21:56:38]

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

[Theresa, 2016-12-06 22:00:48]

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

[Theresa, 2016-12-06 22:00:48]

ptal - this one is reliant on the other patch I sent you. No rush.

[commit-bot: I haz the power, 2016-12-06 23:59:13]

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

[commit-bot: I haz the power, 2016-12-06 23:59:14]

Dry run: This issue passed the CQ dry run.

[gone, 2016-12-07 22:06:20]

lgtm

https://codereview.chromium.org/2553253002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/history/HistoryItemView.java
(right):

https://codereview.chromium.org/2553253002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/history/HistoryItemView.java:36:
mItem = item;
Does it make sense to add a getItem() to the SelectableItemView?  It'd avoid
having to store a second copy of it here.

https://codereview.chromium.org/2553253002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/history/HistoryManager.java
(right):

https://codereview.chromium.org/2553253002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/history/HistoryManager.java:107:
Intent viewIntent = new Intent();
new Intent(Intent.ACTION_VIEW, Uri.parse(url))?

[Theresa, 2016-12-07 23:56:49]

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

[Theresa, 2016-12-07 23:56:51]

https://codereview.chromium.org/2553253002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/history/HistoryItemView.java
(right):

https://codereview.chromium.org/2553253002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/history/HistoryItemView.java:36:
mItem = item;
On 2016/12/07 22:06:20, dfalcantara (check my queue) wrote:
> Does it make sense to add a getItem() to the SelectableItemView?  It'd avoid
> having to store a second copy of it here.

Done.

https://codereview.chromium.org/2553253002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/history/HistoryManager.java
(right):

https://codereview.chromium.org/2553253002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/history/HistoryManager.java:107:
Intent viewIntent = new Intent();
On 2016/12/07 22:06:20, dfalcantara (check my queue) wrote:
> new Intent(Intent.ACTION_VIEW, Uri.parse(url))?

Done.

[commit-bot: I haz the power, 2016-12-07 23:57:20]

Dry run: 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-12-08 01:24:49]

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

[commit-bot: I haz the power, 2016-12-08 01:24:50]

Dry run: This issue passed the CQ dry run.

[Theresa, 2016-12-08 01:58:55]

The CQ bit was checked by twellington@chromium.org

[Theresa, 2016-12-08 01:58:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
dfalcantara@chromium.org
Link to the patchset: https://codereview.chromium.org/2553253002/#ps20001
(title: "Changes from dfalcantara@ review")

[commit-bot: I haz the power, 2016-12-08 01:59:36]

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-12-08 02:06:59]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481162335911970,
"parent_rev": "458443ba7ac96b851df8493618372ac72ad2e6de", "commit_rev":
"20669bacb049dccc084d557d6c0f731c93e54a93"}

[commit-bot: I haz the power, 2016-12-08 02:07:29]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-08 02:12:27]

Description was changed from

==========
[Android History] Add support for opening items

BUG=654071
==========

to

==========
[Android History] Add support for opening items

BUG=654071

Committed: https://crrev.com/3ba7a94420937d5ee40e62afbb0d730a7f751f89
Cr-Commit-Position: refs/heads/master@{#437140}
==========

[commit-bot: I haz the power, 2016-12-08 02:12:30]

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