Run the certificate management dialog with BaseShellDialogImpl.

chrome/browser/ui/webui/options/advanced_options_utils_win.cc

 // Copyright (c) 2012 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/options/advanced_options_utils.h"
 
 #include <windows.h>
 #include <cryptuiapi.h>
 #pragma comment(lib, "cryptui.lib")
 #include <shellapi.h>
 
 #include "base/bind.h"
+#include "base/bind_helpers.h"
+#include "base/location.h"
+#include "base/memory/ref_counted.h"
+#include "base/message_loop/message_loop.h"
 #include "base/path_service.h"
+#include "base/single_thread_task_runner.h"
 #include "base/threading/thread.h"
 #include "chrome/browser/browser_process.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_contents.h"
+#include "ui/shell_dialogs/base_shell_dialog_win.h"
 #include "ui/views/win/hwnd_util.h"
 
 using content::BrowserThread;
 using content::WebContents;
 
 namespace options {
 
+namespace {
+
+// Shows a Windows certificate management dialog on the dialog thread.
+class ManageCertificatesDialog : public ui::BaseShellDialogImpl {
+ public:
+  explicit ManageCertificatesDialog(WebContents* web_contents)
+      : ui_task_runner_(base::MessageLoopForUI::current()->task_runner()) {
+    HWND parent =
+        views::HWNDForNativeWindow(web_contents->GetTopLevelNativeWindow());
+    run_state_ = BeginRun(parent);

[mmenke, 2015/07/23 20:34:29]

Now this seems a bit weird...It really should be in Show(), and EndRun should be
called just before invoking the callback, rather than in the destructor (Yea, I
suggested putting it there in the first place, but now it seems weird.  Other
dialogs seem designed for re-use, too)

[davidben, 2015/07/24 22:43:35]

Done.

+  }
+
+  ~ManageCertificatesDialog() override { EndRun(run_state_); }
+
+  // Shows the dialog and calls |callback| when the dialog closes. The caller
+  // must ensure the ManageCertificatesDialog remains valid until then.
+  void Show(const base::Closure& callback) {

[mmenke, 2015/07/23 20:34:29]

Should we check if (IsRunningDialogForOwner) and just invoke the callback in
that case, or something?  Showing two modal dialogs at once just seems weird.

[davidben, 2015/07/24 22:43:35]

Done.

+    run_state_.dialog_thread->task_runner()->PostTaskAndReply(
+        FROM_HERE, base::Bind(&ManageCertificatesDialog::ShowOnDialogThread,
+                              base::Unretained(this)),
+        callback);
+  }
+
+ private:
+  void ShowOnDialogThread() {
+    CRYPTUI_CERT_MGR_STRUCT cert_mgr = {0};
+    cert_mgr.dwSize = sizeof(CRYPTUI_CERT_MGR_STRUCT);
+    cert_mgr.hwndParent = run_state_.owner;
+    ::CryptUIDlgCertMgr(&cert_mgr);
+    DisableOwner(run_state_.owner);
+  }
+
+  scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner_;

[mmenke, 2015/07/23 20:34:29]

No longer needed.

[davidben, 2015/07/24 22:43:35]

Done.

+  RunState run_state_;
+
+  DISALLOW_COPY_AND_ASSIGN(ManageCertificatesDialog);
+};
+
+}  // namespace
+
 // Callback that opens the Internet Options control panel dialog with the
 // Connections tab selected.
 void OpenConnectionDialogCallback() {
   // Using rundll32 seems better than LaunchConnectionDialog which causes a
   // new dialog to be made for each call.  rundll32 uses the same global
   // dialog and it seems to share with the shortcut in control panel.
   base::FilePath rundll32;
   PathService::Get(base::DIR_SYSTEM, &rundll32);
   rundll32 = rundll32.AppendASCII("rundll32.exe");
 
@@ skipping 16 matching lines @@
 void AdvancedOptionsUtilities::ShowNetworkProxySettings(
       WebContents* web_contents) {
   DCHECK(BrowserThread::IsMessageLoopValid(BrowserThread::FILE));
   BrowserThread::PostTask(BrowserThread::FILE,
                           FROM_HERE,
                           base::Bind(&OpenConnectionDialogCallback));
 }
 
 void AdvancedOptionsUtilities::ShowManageSSLCertificates(
       WebContents* web_contents) {
-  CRYPTUI_CERT_MGR_STRUCT cert_mgr = { 0 };
-  cert_mgr.dwSize = sizeof(CRYPTUI_CERT_MGR_STRUCT);
-  cert_mgr.hwndParent = views::HWNDForNativeWindow(
-      web_contents->GetTopLevelNativeWindow());
-  ::CryptUIDlgCertMgr(&cert_mgr);
+  ManageCertificatesDialog* dialog = new ManageCertificatesDialog(web_contents);
+  dialog->Show(base::Bind(&base::DeletePointer<ManageCertificatesDialog>,
+                          dialog));
 }
 
 }  // namespace options