Plausible fix for crash in GetCertificateChain.

chrome/android/java/src/org/chromium/chrome/browser/WebsiteSettingsPopup.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;
 
 import android.app.Dialog;
 import android.content.Context;
 import android.content.DialogInterface;
 import android.content.Intent;
 import android.graphics.Color;
 import android.provider.Browser;
 import android.text.Html;
 import android.text.TextUtils;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.View.OnClickListener;
 import android.view.ViewGroup.LayoutParams;
 import android.view.Window;
 import android.widget.ImageView;
 import android.widget.LinearLayout;
 import android.widget.ScrollView;
 import android.widget.TextView;
 
 import org.chromium.base.CalledByNative;
 import org.chromium.chrome.R;
-import org.chromium.content.browser.ContentViewCore;
+import org.chromium.content_public.browser.WebContents;
 
 import java.net.URISyntaxException;
 
 /**
  * Java side of Android implementation of the website settings UI.
  */
-class WebsiteSettingsPopup implements OnClickListener {
+public class WebsiteSettingsPopup implements OnClickListener {
     private static final String HELP_URL =
             "http://www.google.com/support/chrome/bin/answer.py?answer=95617";
     private final Context mContext;
     private final Dialog mDialog;
     private final LinearLayout mContainer;
-    private final ContentViewCore mContentViewCore;
+    private final WebContents mWebContents;
     private final int mPadding;
     private TextView mCertificateViewer, mMoreInfoLink;
     private String mLinkUrl;
 
-    private WebsiteSettingsPopup(Context context, ContentViewCore contentViewCore,
-            final long nativeWebsiteSettingsPopup) {
+    private WebsiteSettingsPopup(Context context, WebContents webContents) {
         mContext = context;
+        mWebContents = webContents;
+
+        mContainer = new LinearLayout(mContext);
+        mContainer.setOrientation(LinearLayout.VERTICAL);
+        mPadding = (int) context.getResources().getDimension(R.dimen.certificate_viewer_padding);
+        mContainer.setPadding(mPadding, 0, mPadding, 0);
+
         mDialog = new Dialog(mContext);
         mDialog.requestWindowFeature(Window.FEATURE_NO_TITLE);
         mDialog.setCanceledOnTouchOutside(true);
-        mDialog.setOnCancelListener(new DialogInterface.OnCancelListener() {
+        // This needs to come after other member initialization.
+        final long nativeWebsiteSettingsPopup = nativeInit(this, webContents);
+        mDialog.setOnDismissListener(new DialogInterface.OnDismissListener() {
             @Override
-            public void onCancel(DialogInterface dialogInterface) {
+            public void onDismiss(DialogInterface dialog) {
                 assert nativeWebsiteSettingsPopup != 0;
                 nativeDestroy(nativeWebsiteSettingsPopup);
             }
         });
-        mContainer = new LinearLayout(mContext);
-        mContainer.setOrientation(LinearLayout.VERTICAL);
-        mContentViewCore = contentViewCore;
-        mPadding = (int) context.getResources().getDimension(R.dimen.certificate_viewer_padding);
-        mContainer.setPadding(mPadding, 0, mPadding, 0);
     }
 
     /** Adds a section, which contains an icon, a headline, and a description. */
     @CalledByNative
     private void addSection(int enumeratedIconId, String headline, String description) {
         View section = LayoutInflater.from(mContext).inflate(R.layout.website_settings, null);
         ImageView i = (ImageView) section.findViewById(R.id.website_settings_icon);
         int drawableId = ResourceId.mapToDrawableId(enumeratedIconId);
         i.setImageResource(drawableId);
 
@@ skipping 38 matching lines @@
         mMoreInfoLink = new TextView(mContext);
         mLinkUrl = url;
         mMoreInfoLink.setText(Html.fromHtml("<a href='#'>" + label + "</a>"));
         mMoreInfoLink.setPadding(0, mPadding, 0, mPadding);
         mMoreInfoLink.setOnClickListener(this);
         mContainer.addView(mMoreInfoLink);
     }
 
     /** Displays the WebsiteSettingsPopup. */
     @CalledByNative
-    private void show() {
+    private void showDialog() {
         ScrollView scrollView = new ScrollView(mContext);
         scrollView.addView(mContainer);
         mDialog.addContentView(scrollView,
                 new LinearLayout.LayoutParams(LinearLayout.LayoutParams.MATCH_PARENT,
                         LinearLayout.LayoutParams.MATCH_PARENT));
         mDialog.show();
     }
 
     @Override
     public void onClick(View v) {
         mDialog.dismiss();
         if (mCertificateViewer == v) {
-            byte[][] certChain = nativeGetCertificateChain(mContentViewCore);
+            byte[][] certChain = nativeGetCertificateChain(mWebContents);
+            if (certChain == null) {
+                // The WebContents may have been destroyed/invalidated. If so,
+                // ignore this request.
+                return;
+            }
             CertificateViewer.showCertificateChain(mContext, certChain);
         } else if (mMoreInfoLink == v) {
             try {
                 Intent i = Intent.parseUri(mLinkUrl, Intent.URI_INTENT_SCHEME);
                 i.putExtra(Browser.EXTRA_CREATE_NEW_TAB, true);
                 i.putExtra(Browser.EXTRA_APPLICATION_ID, mContext.getPackageName());
                 mContext.startActivity(i);
             } catch (URISyntaxException ex) {}
         }
     }
 
-    @CalledByNative
-    private static WebsiteSettingsPopup create(Context context, ContentViewCore contentViewCore,
-            long nativeWebsiteSettingsPopup) {
-        return new WebsiteSettingsPopup(context, contentViewCore, nativeWebsiteSettingsPopup);
+    /**
+     * Shows a WebsiteSettings 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.
+     */
+    @SuppressWarnings("unused")
+    public static void show(Context context, WebContents webContents) {
+        new WebsiteSettingsPopup(context, webContents);
     }
 
+    private static native long nativeInit(WebsiteSettingsPopup popup, WebContents webContents);
     private native void nativeDestroy(long nativeWebsiteSettingsPopupAndroid);
-    private native byte[][] nativeGetCertificateChain(ContentViewCore contentViewCore);
-}
+    private native byte[][] nativeGetCertificateChain(WebContents webContents);
+}