UI polish for certificate viewer

chrome/browser/ui/webui/certificate_viewer_webui.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/browser/ui/webui/certificate_viewer_webui.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/i18n/time_formatting.h"
 #include "base/utf_string_conversions.h"
 #include "base/string_number_conversions.h"
 #include "chrome/browser/certificate_viewer.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/certificate_dialogs.h"
+#include "chrome/browser/ui/dialog_style.h"
 #include "chrome/browser/ui/webui/chrome_web_ui.h"
 #include "chrome/common/net/x509_certificate_model.h"
 #include "chrome/common/url_constants.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "grit/generated_resources.h"
 
 #if defined(USE_AURA)
 #include "chrome/browser/ui/constrained_window.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
@@ skipping 48 matching lines @@
 CertificateViewerDialog::~CertificateViewerDialog() {
 }
 
 void CertificateViewerDialog::Show(gfx::NativeWindow parent) {
   // TODO(oshima): Should get browser from parent.
   Browser* browser = BrowserList::GetLastActive();
   DCHECK(browser);
 #if defined(USE_AURA)
   TabContentsWrapper* current_wrapper =
       browser->GetSelectedTabContentsWrapper();
+  // TODO(bshe): UI tweaks needed for AURA html Dialog, such as add padding on
+  // title for AURA ConstrainedHtmlDialog.
   window_ = ConstrainedHtmlUI::CreateConstrainedHtmlDialog(
       current_wrapper->profile(),
       this,
       current_wrapper)->window()->GetNativeWindow();
+#elif defined(OS_CHROMEOS)
+  window_ = browser->BrowserShowHtmlDialog(this, parent,
+      static_cast<DialogStyle>(STYLE_XBAR | STYLE_FLUSH_CONTENT));
 #else
-  window_ = browser->BrowserShowHtmlDialog(this, parent);
+  window_ = browser->BrowserShowHtmlDialog(this, parent, STYLE_GENERIC);
 #endif
 }
 
 bool CertificateViewerDialog::IsDialogModal() const {
   return false;
 }
 
 string16 CertificateViewerDialog::GetDialogTitle() const {
   return title_;
 }
@@ skipping 313 matching lines @@
   cert_sub_fields->Append(node_details = new DictionaryValue());
   node_details->SetString("label",
       l10n_util::GetStringUTF8(IDS_CERT_INFO_SHA1_FINGERPRINT_LABEL));
   node_details->SetString("payload.val",
       x509_certificate_model::HashCertSHA1(cert));
 
   // Send certificate information to javascript.
   web_ui_->CallJavascriptFunction("cert_viewer.getCertificateFields",
       root_list);
 }