Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1077)

Unified Diff: chrome/browser/ui/android/website_settings_popup_android.h

Issue 330583002: Plausible fix for crash in GetCertificateChain. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/android/website_settings_popup_android.h
diff --git a/chrome/browser/ui/android/website_settings_popup_android.h b/chrome/browser/ui/android/website_settings_popup_android.h
index ea9e5b2f56a64a07f7477bdd5b0325f3442862ac..5ba6207d2f8b4e21348e5f9e9abe99283243f4fb 100644
--- a/chrome/browser/ui/android/website_settings_popup_android.h
+++ b/chrome/browser/ui/android/website_settings_popup_android.h
@@ -16,14 +16,12 @@ namespace content {
class WebContents;
}
-// Androidimplementation of the website settings UI.
+// Android implementation of the website settings UI.
class WebsiteSettingsPopupAndroid : public WebsiteSettingsUI {
public:
- static void Show(JNIEnv* env,
- jobject context,
- jobject java_content_view,
- content::WebContents* web_contents);
-
+ WebsiteSettingsPopupAndroid(JNIEnv* env,
+ jobject java_website_settings,
+ content::WebContents* web_contents);
virtual ~WebsiteSettingsPopupAndroid();
void Destroy(JNIEnv* env, jobject obj);
@@ -38,11 +36,6 @@ class WebsiteSettingsPopupAndroid : public WebsiteSettingsUI {
static bool RegisterWebsiteSettingsPopupAndroid(JNIEnv* env);
private:
- WebsiteSettingsPopupAndroid(JNIEnv* env,
- jobject context,
- jobject java_content_view,
- content::WebContents* web_contents);
-
// The presenter that controlls the Website Settings UI.
scoped_ptr<WebsiteSettings> presenter_;

Powered by Google App Engine
This is Rietveld 408576698