Factor getCertificateChain out of ConnectionPopupInfo

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

Index: chrome/android/java/src/org/chromium/chrome/browser/pageinfo/ConnectionInfoPopup.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/pageinfo/ConnectionInfoPopup.java b/chrome/android/java/src/org/chromium/chrome/browser/pageinfo/ConnectionInfoPopup.java
index b99c9289cf39f4ef698b1414c2e7c99c304d2893..6853d9bfee862f95220395d23142a1b720e61c42 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/pageinfo/ConnectionInfoPopup.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/pageinfo/ConnectionInfoPopup.java
@@ -228,6 +228,10 @@ public class ConnectionInfoPopup implements OnClickListener {
         }
     }
 
+    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.
      *
@@ -247,5 +251,5 @@ public class ConnectionInfoPopup implements OnClickListener {
     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);
 }