Fix minor WebUI dialog issues, mostly with bookmark management

chrome/browser/ui/webui/certificate_viewer.cc

Index: chrome/browser/ui/webui/certificate_viewer.cc
diff --git a/chrome/browser/ui/webui/certificate_viewer.cc b/chrome/browser/ui/webui/certificate_viewer.cc
index 22cb52e57940b50db6f5c51df34ff320278d0750..be920123fe913a519a95efa6ae78e3326fe7b2f7 100644
--- a/chrome/browser/ui/webui/certificate_viewer.cc
+++ b/chrome/browser/ui/webui/certificate_viewer.cc
@@ -69,7 +69,7 @@ CertificateViewerDialog::~CertificateViewerDialog() {
 }
 
 bool CertificateViewerDialog::IsDialogModal() const {
-  return true;
+  return false;

[flackr, 2011/10/24 19:16:01]

This might allow a crash when exporting a certificate from a certificate viewer
dialog whose parent window has been closed. This is because currently
CertificateViewerDialogHandler::ExportCertificate passes the parent of the
certificate viewer dialog through to ShowCertExportDialog. You can assign this
to me to fix if you'd like.

[Rick Byers, 2011/10/24 20:02:42]

Ah, good catch.  Do you think the fix is easy?  I'd rather not block UI review
on this, so I've filed http://code.google.com/p/chromium/issues/detail?id=101402
to you.  However, if you think a better option is to just make the certificate
viewer modal (i.e. if there isn't an easy fix), then we could go with that to UI
review instead.  

[flackr, 2011/10/24 20:39:33]

I think the fix will be easy. There has to be some window that we can select and
parent the dialog on even if not the certificate viewer dialog window.

 }
 
 string16 CertificateViewerDialog::GetDialogTitle() const {