[Android] Fix 'open in browser' in CCT does not work for images in Google image search

chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java

 // Copyright 2013 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.
 
 package org.chromium.chrome.browser.contextmenu;
 
 import android.content.Context;
 import android.net.MailTo;
 import android.support.annotation.IntDef;
 import android.support.annotation.StringRes;
@@ skipping 598 matching lines @@
      * @return True if the url is empty or "about:blank".
      */
     private boolean isEmptyUrl(String url) {
         if (TextUtils.isEmpty(url) || url.equals(ContentUrlConstants.ABOUT_BLANK_DISPLAY_URL)) {
             return true;
         }
         return false;
     }
 
     /**
-     * The valid url of a link is stored in the linkUrl of ContextMenuParams while the
-     * valid url of a image or video is stored in the srcUrl of ContextMenuParams.
-     * @param params The parameters used to decide the type of the content.
+     * Return the valid url of a ContextMenuParams.
+     * If the ContextMenuParams is an anchor and its linkUrl is not empty, returns the linkUrl.
+     * Otherwise returns the srcUrl.
+     * @param params The {@link ContextMenuParams} to check.
      */
     private String getUrl(ContextMenuParams params) {
-        if (params.isImage() || params.isVideo()) {
+        if (params.isAnchor() && !isEmptyUrl(params.getLinkUrl())) {

[Ted C, 2017/05/19 16:55:07]

per offline discussion, let's remove the isEmptyUrl check at all.

[ltian, 2017/05/19 18:02:04]

Done.

+            return params.getLinkUrl();
+        } else {
             return params.getSrcUrl();
-        } else {
-            return params.getLinkUrl();
         }
     }
 }