Don't select a client certs for TabContents with no TabContentsWrapper.

chrome/browser/chrome_content_browser_client.cc

Index: chrome/browser/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index 630d9b96c1c3a260587fabdd4dc8408bdcc73dd7..4bdb72724ec0e33ca68bab2fe43fc1dceceb7199 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -856,7 +856,11 @@ void ChromeContentBrowserClient::SelectClientCertificate(
 
   TabContentsWrapper* wrapper =
       TabContentsWrapper::GetCurrentWrapperForContents(tab);
-  wrapper->ssl_helper()->SelectClientCertificate(handler);
+  if (wrapper) {

[Mattias Nissler (ping if slow), 2011/11/30 19:56:16]

no need for curlies here.

[markusheintz_, 2011/12/02 14:54:20]

I think the style guide is ok with that if you have an else. 

[markusheintz_, 2011/12/02 14:54:20]

I guess the style quide is fine with that.

+    wrapper->ssl_helper()->SelectClientCertificate(handler);
+  } else {
+    handler->CertificateSelected(NULL);

[wtc, 2011/11/30 23:59:43]

It seems that what we do in this case should match what we
do above on lines 853-854 if |tab| is NULL.

[markusheintz_, 2011/12/02 14:54:20]

What happens to the request if we don't call CertificateSelected(NULL) on the
handler? Does it have to time out?

I can not add a NOTREACHED, this would crash chrome every time a background page
of an extension send a xhr to a site that requests a client cert.

+  }
 }
 
 void ChromeContentBrowserClient::AddNewCertificate(