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

ios/chrome/browser/ui/browser_view_controller.mm

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "ios/chrome/browser/ui/browser_view_controller.h"
 
 #import <AssetsLibrary/AssetsLibrary.h>
 #import <MobileCoreServices/MobileCoreServices.h>
 #import <PassKit/PassKit.h>
 #import <Photos/Photos.h>
@@ skipping 2430 matching lines @@
     if (link.SchemeIs(url::kJavaScriptScheme)) {
       // Open
       title = l10n_util::GetNSStringWithFixup(IDS_IOS_CONTENT_CONTEXT_OPEN);
       action = ^{
         Record(ACTION_OPEN_JAVASCRIPT, isImage, isLink);
         [weakSelf openJavascript:base::SysUTF8ToNSString(link.GetContent())];
       };
       [_contextMenuCoordinator addItemWithTitle:title action:action];
     }
 
     if (web::UrlHasWebScheme(link)) {

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

I know this is not in the scope of this CL, but this test is too permissive for
ReadingList.
It should be SchemeIsHTTPOrHTTPS.

       web::Referrer referrer([_model currentTab].url, params.referrer_policy);
 
-      if (reading_list::switches::IsReadingListEnabled()) {
-        NSString* innerText = params.link_text;
-        if ([innerText length] > 0) {
-          // Add to reading list.
-          title = l10n_util::GetNSStringWithFixup(
-              IDS_IOS_CONTENT_CONTEXT_ADDTOREADINGLIST);
-          action = ^{
-            Record(ACTION_READ_LATER, isImage, isLink);
-            [weakSelf addToReadingListURL:link title:innerText];
-          };
-          [_contextMenuCoordinator addItemWithTitle:title action:action];
-        }
-      }
-
       // Open in New Tab.
       title = l10n_util::GetNSStringWithFixup(
           IDS_IOS_CONTENT_CONTEXT_OPENLINKNEWTAB);
       action = ^{
         Record(ACTION_OPEN_IN_NEW_TAB, isImage, isLink);
         [weakSelf webPageOrderedOpen:link
                             referrer:referrer
                           windowName:nil
                         inBackground:YES
                             appendTo:kCurrentTab];
       };
       [_contextMenuCoordinator addItemWithTitle:title action:action];
       if (!_isOffTheRecord) {
         // Open in Incognito Tab.
         title = l10n_util::GetNSStringWithFixup(
             IDS_IOS_CONTENT_CONTEXT_OPENLINKNEWINCOGNITOTAB);
         action = ^{
           Record(ACTION_OPEN_IN_INCOGNITO_TAB, isImage, isLink);
           [weakSelf webPageOrderedOpen:link
                               referrer:referrer
                             windowName:nil
                            inIncognito:YES
                           inBackground:NO
                               appendTo:kCurrentTab];
         };
         [_contextMenuCoordinator addItemWithTitle:title action:action];
       }
+
+      if (reading_list::switches::IsReadingListEnabled()) {
+        NSString* innerText = params.link_text;
+        if ([innerText length] > 0) {
+          // Add to reading list.
+          title = l10n_util::GetNSStringWithFixup(
+              IDS_IOS_CONTENT_CONTEXT_ADDTOREADINGLIST);
+          action = ^{
+            Record(ACTION_READ_LATER, isImage, isLink);
+            [weakSelf addToReadingListURL:link title:innerText];
+          };
+          [_contextMenuCoordinator addItemWithTitle:title action:action];
+        }
+      }
     }
     // Copy Link.
     title = l10n_util::GetNSStringWithFixup(IDS_IOS_CONTENT_CONTEXT_COPY);
     action = ^{
       Record(ACTION_COPY_LINK_ADDRESS, isImage, isLink);
       NSURL* url = net::NSURLWithGURL(link);
       NSDictionary* item = @{
         (NSString*)kUTTypeURL : url,
         (NSString*)kUTTypeUTF8PlainText :
             [[url absoluteString] dataUsingEncoding:NSUTF8StringEncoding],
@@ skipping 2637 matching lines @@
 
 - (UIView*)voiceSearchButton {
   return _voiceSearchButton;
 }
 
 - (id<LogoAnimationControllerOwner>)logoAnimationControllerOwner {
   return [self currentLogoAnimationControllerOwner];
 }
 
 @end