UI polish for certificate viewer

chrome/browser/ui/webui/certificate_viewer.h

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 add it to the window creation
call.

[bshe, 2011/11/21 16:09:37]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(CertificateViewerDialog);
 };