[Android] 'Open in browser' in context menu for CCT does not work for image.

[Android] 'Open in browser' in context menu for CCT does not work for image.

When the default browser is not set, the 'Open in browser' option in the
context menu for CCT does not work for image and video link.

This is because for image and video, the linkUrl of the
ContextMenuParams is null. It is a same problem as the bug here:
https://bugs.chromium.org/p/chromium/issues/detail?id=696417.

And to fix it, follow this CL:
https://codereview.chromium.org/2739723004/ that checks the type of link
first then passes different urls for different types of link.

BUG=706209
Review-Url: https://codereview.chromium.org/2779113006
Cr-Commit-Position: refs/heads/master@{#460505}
Committed: https://chromium.googlesource.com/chromium/src/+/83bb552f1111fc669d47cb8070f647a8e2caf1d7

[ltian, 2017-03-29 19:01:53]

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

[ltian, 2017-03-29 19:01:53]

dfalcantara@chromium.org: Can you take a look for this CL?
 It has the similar problem as this CL
(https://codereview.chromium.org/2739723004/) which I forgot to update it every
place. Sorry for that!

Thanks!

[gone, 2017-03-29 19:04:01]

lgtm

[ltian, 2017-03-29 19:04:29]

The CQ bit was checked by ltian@chromium.org

[commit-bot: I haz the power, 2017-03-29 19:05:39]

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-03-29 20:00:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490814269019810, "parent_rev":
"40e8020cbd6c9920cef435a68d5c38addff57c6a", "commit_rev":
"83bb552f1111fc669d47cb8070f647a8e2caf1d7"}

[commit-bot: I haz the power, 2017-03-29 20:01:32]

Description was changed from

==========
[Android] 'Open in browser' in context menu for CCT does not work for image.

When the default browser is not set, the 'Open in browser' option in the
context menu for CCT does not work for image and video link.

This is because for image and video, the linkUrl of the
ContextMenuParams is null. It is a same problem as the bug here:
https://bugs.chromium.org/p/chromium/issues/detail?id=696417.

And to fix it, follow this CL:
https://codereview.chromium.org/2739723004/ that checks the type of link
first then passes different urls for different types of link.

BUG=706209
==========

to

==========
[Android] 'Open in browser' in context menu for CCT does not work for image.

When the default browser is not set, the 'Open in browser' option in the
context menu for CCT does not work for image and video link.

This is because for image and video, the linkUrl of the
ContextMenuParams is null. It is a same problem as the bug here:
https://bugs.chromium.org/p/chromium/issues/detail?id=696417.

And to fix it, follow this CL:
https://codereview.chromium.org/2739723004/ that checks the type of link
first then passes different urls for different types of link.

BUG=706209

Review-Url: https://codereview.chromium.org/2779113006
Cr-Commit-Position: refs/heads/master@{#460505}
Committed:
https://chromium.googlesource.com/chromium/src/+/83bb552f1111fc669d47cb8070f6...
==========

[commit-bot: I haz the power, 2017-03-29 20:01:34]

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