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

chrome/browser/ssl/ssl_client_auth_requestor_mock.h

+// 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.
+
+#ifndef CHROME_BROWSER_SSL_SSL_CLIENT_AUTH_REQUESTOR_MOCK_H_
+#define CHROME_BROWSER_SSL_SSL_CLIENT_AUTH_REQUESTOR_MOCK_H_
+#pragma once
+
+#include "base/memory/ref_counted.h"
+#include "net/http/http_transaction_factory.h"
+#include "net/url_request/url_request.h"
+#include "net/url_request/url_request_context.h"
+#include "testing/gmock/include/gmock/gmock.h"
+
+namespace net {
+class SSLCertRequestInfo;
+class X509Certificate;
+}
+
+class SSLClientAuthRequestorMock

[Avi (use Gerrit), 2012/02/13 20:10:37]

"Requestor" is not English. Yet
http://code.google.com/codesearch#search/&exact_package=chromium&q=requestor
returns lots of results. Can you point me to a source of enlightenment?

[jam, 2012/02/13 20:59:00]

I didn't think about it that much :) It looks like our code uses both, and
they're interchangeable per http://www.merriam-webster.com/dictionary/requestor.
if you want i can rename this to requester

+    : public base::RefCountedThreadSafe<SSLClientAuthRequestorMock> {
+ public:
+  SSLClientAuthRequestorMock(
+      net::URLRequest* request,
+      net::SSLCertRequestInfo* cert_request_info)
+      : cert_request_info_(cert_request_info),
+        http_network_session_(
+            request->context()->http_transaction_factory()->GetSession()) {
+  }
+  // NOTE: we need a vtable or else gmock blows up.
+  virtual ~SSLClientAuthRequestorMock() {
+  }
+
+  MOCK_METHOD1(CertificateSelected, void(net::X509Certificate* cert));
+
+  net::SSLCertRequestInfo* cert_request_info_;
+  net::HttpNetworkSession* http_network_session_;
+};
+
+#endif  // CHROME_BROWSER_SSL_SSL_CLIENT_AUTH_REQUESTOR_MOCK_H_