Remove knowledge about SSLClientAuthHandler from chrome. Instead a callback is given to the embed...

chrome/browser/ui/webui/ssl_client_certificate_selector_webui.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/ssl_client_certificate_selector_webui.h"
 
 #include <vector>
 #include <string>
 
 #include "base/bind.h"
@@ skipping 97 matching lines @@
   return rv;
 }
 
 }  // namespace
 
 ////////////////////////////////////////////////////////////////////////////////
 // SSLClientCertificateSelectorWebUI
 
 void SSLClientCertificateSelectorWebUI::ShowDialog(
     TabContentsWrapper* wrapper,
+    const net::HttpNetworkSession* network_session,
     net::SSLCertRequestInfo* cert_request_info,
-    SSLClientAuthHandler* delegate) {
+    const base::Callback<void(net::X509Certificate*)>& callback) {
   SSLClientCertificateSelectorWebUI* ui =
       new SSLClientCertificateSelectorWebUI(wrapper,
+                                            network_session,
                                             cert_request_info,
-                                            delegate);
+                                            callback);
   Profile* profile = wrapper->profile();
   ConstrainedHtmlUI::CreateConstrainedHtmlDialog(profile, ui, NULL, wrapper);
 }
 
 SSLClientCertificateSelectorWebUI::SSLClientCertificateSelectorWebUI(
     TabContentsWrapper* wrapper,
+    const net::HttpNetworkSession* network_session,
     net::SSLCertRequestInfo* cert_request_info,
-    SSLClientAuthHandler* delegate)
-    : wrapper_(wrapper),
-      cert_request_info_(cert_request_info),
-      delegate_(delegate) {
+    const base::Callback<void(net::X509Certificate*)>& callback)
+    : SSLClientAuthObserver(network_session, cert_request_info, callback),
+      wrapper_(wrapper) {
 
   ChromeWebUIDataSource* source = new ChromeWebUIDataSource(
       chrome::kChromeUISSLClientCertificateSelectorHost);
 
   source->AddLocalizedString("siteLabel",
                              IDS_CERT_SELECTOR_SITE_DESCRIPTION_LABEL);
   source->AddLocalizedString("chooseLabel",
                              IDS_CERT_SELECTOR_CHOOSE_DESCRIPTION_LABEL);
   source->AddLocalizedString("detailsLabel",
                              IDS_CERT_SELECTOR_DETAILS_DESCRIPTION_LABEL);
@@ skipping 12 matching lines @@
 
   // Set default resource.
   source->set_default_resource(IDR_SSL_CLIENT_CERTIFICATE_SELECTOR_HTML);
 
   wrapper->profile()->GetChromeURLDataManager()->AddDataSource(source);
 }
 
 SSLClientCertificateSelectorWebUI::~SSLClientCertificateSelectorWebUI() {
 }
 
+void SSLClientCertificateSelectorWebUI::OnCertSelectedByNotification() {
+  NOTIMPLEMENTED();
+}
+
 // HtmlDialogUIDelegate methods
 ui::ModalType SSLClientCertificateSelectorWebUI::GetDialogModalType() const {
   return ui::MODAL_TYPE_WINDOW;
 }
 
 string16 SSLClientCertificateSelectorWebUI::GetDialogTitle() const {
   return string16();
 }
 
 GURL SSLClientCertificateSelectorWebUI::GetDialogContentURL() const {
@@ skipping 15 matching lines @@
 
 void SSLClientCertificateSelectorWebUI::OnDialogClosed(
     const std::string& json_retval) {
   scoped_ptr<Value> parsed_value(base::JSONReader::Read(json_retval, false));
   ListValue* list = NULL;
   double value = 0;
   if (parsed_value.get() && parsed_value->GetAsList(&list) &&
       list->GetDouble(0, &value)) {
     // User selected a certificate.
     size_t index = (size_t) value;
-    if (index < cert_request_info_->client_certs.size()) {
+    if (index < cert_request_info()->client_certs.size()) {
       scoped_refptr<net::X509Certificate> selected_cert(
-          cert_request_info_->client_certs[index]);
+          cert_request_info()->client_certs[index]);
       browser::UnlockCertSlotIfNecessary(
           selected_cert,
           browser::kCryptoModulePasswordClientAuth,
-          cert_request_info_->host_and_port,
-          base::Bind(&SSLClientCertificateSelectorWebUI::Unlocked,
-                     delegate_,
-                     selected_cert));
+          cert_request_info()->host_and_port,
+          base::Bind(&SSLClientCertificateSelectorWebUI::CertificateSelected,
+                     base::Unretained(this), selected_cert));
       return;
     }
   }
 
   // Anything else constitutes a cancel.
-  delegate_->CertificateSelected(NULL);
-  delegate_ = NULL;
-}
-
-// static
-void SSLClientCertificateSelectorWebUI::Unlocked(SSLClientAuthHandler* delegate,
-    net::X509Certificate* selected_cert) {
-  delegate->CertificateSelected(selected_cert);
+  CertificateSelected(NULL);
 }
 
 
 void SSLClientCertificateSelectorWebUI::OnCloseContents(WebContents* source,
     bool* out_close_dialog) {
   NOTIMPLEMENTED();
 }
 
 bool SSLClientCertificateSelectorWebUI::ShouldShowDialogTitle() const {
   return true;
 }
 
 // WebUIMessageHandler methods
 
 void SSLClientCertificateSelectorWebUI::RegisterMessages() {
   web_ui()->RegisterMessageCallback("requestDetails",
       base::Bind(&SSLClientCertificateSelectorWebUI::RequestDetails,
                  base::Unretained(this)));
   web_ui()->RegisterMessageCallback("viewCertificate",
       base::Bind(&SSLClientCertificateSelectorWebUI::ViewCertificate,
                  base::Unretained(this)));
 }
 
 void SSLClientCertificateSelectorWebUI::RequestDetails(
     const base::ListValue* args) {
   std::vector<std::string> nicknames;
   x509_certificate_model::GetNicknameStringsFromCertList(
-      cert_request_info_->client_certs,
+      cert_request_info()->client_certs,
       l10n_util::GetStringUTF8(IDS_CERT_SELECTOR_CERT_EXPIRED),
       l10n_util::GetStringUTF8(IDS_CERT_SELECTOR_CERT_NOT_YET_VALID),
       &nicknames);
 
-  DCHECK_EQ(nicknames.size(), cert_request_info_->client_certs.size());
+  DCHECK_EQ(nicknames.size(), cert_request_info()->client_certs.size());
 
   DictionaryValue dict;
-  dict.SetString("site", cert_request_info_->host_and_port.c_str());
+  dict.SetString("site", cert_request_info()->host_and_port.c_str());
   ListValue* certificates = new ListValue();
   ListValue* details = new ListValue();
-  for (size_t i = 0; i < cert_request_info_->client_certs.size(); ++i) {
+  for (size_t i = 0; i < cert_request_info()->client_certs.size(); ++i) {
     net::X509Certificate::OSCertHandle cert =
-        cert_request_info_->client_certs[i]->os_cert_handle();
+        cert_request_info()->client_certs[i]->os_cert_handle();
 
     certificates->Append(new StringValue(
         FormatComboBoxText(cert, nicknames[i])));
     details->Append(new StringValue(
         FormatDetailsText(cert)));
   }
   dict.Set("certificates", certificates);
   dict.Set("details", details);
   // Send list of tab contents details to javascript.
   web_ui()->CallJavascriptFunction(
       "sslClientCertificateSelector.setDetails",
       dict);
 }
 
 void SSLClientCertificateSelectorWebUI::ViewCertificate(
     const base::ListValue* args) {
   double value = 0;
   if (args && args->GetDouble(0, &value)) {
     size_t index = (size_t) value;
-    if (index < cert_request_info_->client_certs.size())
-      ShowCertificateViewer(NULL, cert_request_info_->client_certs[index]);
+    if (index < cert_request_info()->client_certs.size())
+      ShowCertificateViewer(NULL, cert_request_info()->client_certs[index]);
   }
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // Platform API
 
 namespace browser {
 
 void ShowSSLClientCertificateSelector(
     TabContentsWrapper* wrapper,
+    const net::HttpNetworkSession* network_session,
     net::SSLCertRequestInfo* cert_request_info,
-    SSLClientAuthHandler* delegate) {
+    const base::Callback<void(net::X509Certificate*)>& callback) {
   // TODO(jhawkins): move this into a non-webui location.
 #if defined(USE_AURA)
-  ShowNativeSSLClientCertificateSelector(wrapper,
-                                         cert_request_info,
-                                         delegate);
+  ShowNativeSSLClientCertificateSelector(
+      wrapper, network_session, cert_request_info, callback);
 #elif defined(OS_CHROMEOS)
-  SSLClientCertificateSelectorWebUI::ShowDialog(wrapper,
-                                                cert_request_info,
-                                                delegate);
+  SSLClientCertificateSelectorWebUI::ShowDialog(
+      wrapper, network_session, cert_request_info, callback);
 #else
   // TODO(rbyers): Remove the IsMoreWebUI check and (ideally) all #ifdefs onnce
   // we can select exactly one version of this dialog to use for each platform
   // at build time.  http://crbug.com/102775
-  if (chrome_web_ui::IsMoreWebUI())
-    SSLClientCertificateSelectorWebUI::ShowDialog(wrapper,
-                                                  cert_request_info,
-                                                  delegate);
-  else
-    ShowNativeSSLClientCertificateSelector(wrapper,
-                                           cert_request_info,
-                                           delegate);
+  if (chrome_web_ui::IsMoreWebUI()) {
+    SSLClientCertificateSelectorWebUI::ShowDialog(
+        wrapper, network_session, cert_request_info, callback);
+  } else {
+    ShowNativeSSLClientCertificateSelector(
+        wrapper, network_session, cert_request_info, callback);
+  }
 #endif
 }
 
 }  // namespace browser