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)
+      : web_contents_(web_contents) {}

[mmenke, 2015/07/27 18:45:33]

optional:  Should we take the WebContents as a parameter to show instead?

[davidben, 2015/07/27 18:47:45]

I guess the question here is "what is a dialog object?". And, honestly, I have
no clue. BaseShellDialogImpl, despite being something you subclass, has no
actual state.

[mmenke, 2015/07/27 19:08:14]

I agree, and I'm with you on having no clue.

[davidben, 2015/07/27 19:24:19]

ananta? You wrote BaseShellDialogImpl. How are dialog classes in our code
supposed to work? (How much of what should be associated with the class and how
much should be associated with an individual call to Show() and passed along the
base::Binds?)

[ananta, 2015/07/27 19:59:23]

That is upto the caller. Please look at SelectFileDialogImpl for an example. For
this use case, I don't see why Show cannot take the web_contents_ in as an
argument. Alternatively you can pass in the parent HWND directly.

+
+  ~ManageCertificatesDialog() override {}
+
+  // 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) {
+    HWND parent =
+        views::HWNDForNativeWindow(web_contents_->GetTopLevelNativeWindow());
+    if (IsRunningDialogForOwner(parent)) {
+      base::MessageLoop::current()->PostTask(FROM_HERE, callback);
+      return;
+    }
+
+    RunState run_state = BeginRun(parent);
+    run_state.dialog_thread->task_runner()->PostTaskAndReply(
+        FROM_HERE, base::Bind(&ManageCertificatesDialog::ShowOnDialogThread,
+                              base::Unretained(this), run_state),
+        base::Bind(&ManageCertificatesDialog::OnDialogClosed,
+                   base::Unretained(this), run_state, callback));
+  }
+
+ private:
+  void ShowOnDialogThread(const RunState& run_state) {
+    CRYPTUI_CERT_MGR_STRUCT cert_mgr = {0};
+    cert_mgr.dwSize = sizeof(CRYPTUI_CERT_MGR_STRUCT);
+    cert_mgr.hwndParent = run_state.owner;
+    ::CryptUIDlgCertMgr(&cert_mgr);
+  }
+
+  void OnDialogClosed(const RunState& run_state,
+                      const base::Closure& callback) {

[mmenke, 2015/07/27 19:08:14]

Should we have a weak pointer here?  Or this should be static.

[mmenke, 2015/07/27 19:13:30]

Oh, right...with your ownership model, not needed.

+    EndRun(run_state);
+    // May delete |this|.
+    callback.Run();
+  }
+
+  WebContents* web_contents_;
+
+  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