Move "Read Later" to the third place in the context menu

Move "Read Later" to the third place in the context menu

BUG=681627
Review-Url: https://codereview.chromium.org/2631393002
Cr-Commit-Position: refs/heads/master@{#444010}
Committed: https://chromium.googlesource.com/chromium/src/+/5fd4034984b118b932a891b849320c147f47e702

[gambard, 2017-01-17 08:19:08]

gambard@chromium.org changed reviewers:
+ marq@chromium.org

[gambard, 2017-01-17 08:19:08]

PTAL, quick review.

[marq (ping after 24h), 2017-01-17 09:08:42]

lgtm

[gambard, 2017-01-17 09:09:01]

The CQ bit was checked by gambard@chromium.org

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

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 09:17:57]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484644141814830, "parent_rev":
"88a01ea6624dcffe4257a05e79c56ccd5eb31849", "commit_rev":
"5fd4034984b118b932a891b849320c147f47e702"}

[commit-bot: I haz the power, 2017-01-17 09:18:24]

Description was changed from

==========
Move "Read Later" to the third place in the context menu

BUG=681627
==========

to

==========
Move "Read Later" to the third place in the context menu

BUG=681627

Review-Url: https://codereview.chromium.org/2631393002
Cr-Commit-Position: refs/heads/master@{#444010}
Committed:
https://chromium.googlesource.com/chromium/src/+/5fd4034984b118b932a891b84932...
==========

[commit-bot: I haz the power, 2017-01-17 09:18:25]

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

[Olivier, 2017-01-17 09:34:55]

olivierrobin@chromium.org changed reviewers:
+ olivierrobin@chromium.org

[Olivier, 2017-01-17 09:34:55]

https://codereview.chromium.org/2631393002/diff/1/ios/chrome/browser/ui/brows...
File ios/chrome/browser/ui/browser_view_controller.mm (right):

https://codereview.chromium.org/2631393002/diff/1/ios/chrome/browser/ui/brows...
ios/chrome/browser/ui/browser_view_controller.mm:2451: if
(web::UrlHasWebScheme(link)) {
I know this is not in the scope of this CL, but this test is too permissive for
ReadingList.
It should be SchemeIsHTTPOrHTTPS.