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

Unified Diff: chrome/browser/ui/webui/certificate_viewer.h

Issue 8479042: UI polish for certificate viewer (Closed) Base URL: /usr/local/google/home/bshe/NoTouchChromium/../TouchChromium/src/@trunk
Patch Set: Add x button and remove padding and close button for certificate viewer. Created 9 years, 1 month 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/webui/certificate_viewer.h
diff --git a/chrome/browser/ui/webui/certificate_viewer.h b/chrome/browser/ui/webui/certificate_viewer.h
index 93566e0a23dcabc8c0f8e36c08de7fa426be3187..dd2b1027e1c1cb2d5b5bae9bf11f72b76db67aa4 100644
--- a/chrome/browser/ui/webui/certificate_viewer.h
+++ b/chrome/browser/ui/webui/certificate_viewer.h
@@ -11,6 +11,10 @@
#include "net/base/x509_certificate.h"
#include "ui/gfx/native_widget_types.h"
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/frame/bubble_window_style.h"
+#endif
+
// Displays the native or WebUI certificate viewer dialog for the given
// certificate.
void ShowCertificateViewer(gfx::NativeWindow parent,
@@ -31,7 +35,12 @@ class CertificateViewerDialog : private HtmlDialogUIDelegate {
// Construct a certificate viewer for the passed in certificate. A reference
// to the certificate pointer is added for the lifetime of the certificate
// viewer.
+#if defined(OS_CHROMEOS)
+ explicit CertificateViewerDialog(net::X509Certificate* cert,
+ chromeos::BubbleWindowStyle style);
+#else
explicit CertificateViewerDialog(net::X509Certificate* cert);
+#endif
// Show the dialog using the given parent window.
void Show(gfx::NativeWindow parent);
@@ -59,6 +68,10 @@ class CertificateViewerDialog : private HtmlDialogUIDelegate {
// The title of the certificate viewer dialog, Certificate Viewer: CN.
string16 title_;
+#if defined(OS_CHROMEOS)
+ chromeos::BubbleWindowStyle style_;
+#endif
flackr 2011/11/10 16:03:16 Don't add style as a parameter to the class, just
bshe 2011/11/21 16:09:37 Done.
+
DISALLOW_COPY_AND_ASSIGN(CertificateViewerDialog);
};

Powered by Google App Engine
This is Rietveld 408576698