Factor getCertificateChain out of ConnectionPopupInfo

chrome/android/java/src/org/chromium/chrome/browser/pageinfo/ConnectionInfoPopup.java

 // Copyright 2015 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.pageinfo;
 
 import android.app.Dialog;
 import android.content.Context;
 import android.content.DialogInterface;
 import android.content.Intent;
@@ skipping 210 matching lines @@
                 i.putExtra(Browser.EXTRA_CREATE_NEW_TAB, true);
                 i.putExtra(Browser.EXTRA_APPLICATION_ID, mContext.getPackageName());
                 mContext.startActivity(i);
             } catch (Exception ex) {
                 // Do nothing intentionally.
                 Log.w(TAG, "Bad URI %s", mLinkUrl, ex);
             }
         }
     }
 
+    public static byte[][] getCertificateChain(WebContents webContents) {

[please use gerrit instead, 2016/12/09 17:16:01]

It would be best to place this in a separate class that's not related to the
"connection info popup".

+        return nativeGetCertificateChain(webContents);
+    }
+
     /**
      * Shows a connection info dialog for the provided WebContents.
      *
      * The popup adds itself to the view hierarchy which owns the reference while it's
      * visible.
      *
      * @param context Context which is used for launching a dialog.
      * @param webContents The WebContents for which to show Website information. This
      *         information is retrieved for the visible entry.
      */
     public static void show(Context context, WebContents webContents) {
         new ConnectionInfoPopup(context, webContents);
     }
 
     private static native long nativeInit(ConnectionInfoPopup popup,
             WebContents webContents);
     private native void nativeDestroy(long nativeConnectionInfoPopupAndroid);
     private native void nativeResetCertDecisions(
             long nativeConnectionInfoPopupAndroid, WebContents webContents);
-    private native byte[][] nativeGetCertificateChain(WebContents webContents);
+    private static native byte[][] nativeGetCertificateChain(WebContents webContents);
 }