Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4132)

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 124113003: Remove calls of PrerenderTracker::TryCancel and TryCancelOnIOThread in ChromeResourceDispatcherHost… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 6 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chrome_content_browser_client.cc
===================================================================
--- chrome/browser/chrome_content_browser_client.cc (revision 242921)
+++ chrome/browser/chrome_content_browser_client.cc (working copy)
@@ -1815,17 +1815,25 @@
const base::Callback<void(net::X509Certificate*)>& callback) {
content::RenderFrameHost* rfh = content::RenderFrameHost::FromID(
render_process_id, render_frame_id);
- if (!rfh) {
jam 2014/01/03 23:22:10 this is unnecessary since WebContents::FromRenderF
- NOTREACHED();
- return;
- }
-
WebContents* tab = WebContents::FromRenderFrameHost(rfh);
if (!tab) {
NOTREACHED();
return;
}
+ prerender::PrerenderManager* prerender_manager =
+ prerender::PrerenderManagerFactory::GetForProfile(
+ Profile::FromBrowserContext(tab->GetBrowserContext()));
+ if (prerender_manager) {
+ prerender::PrerenderContents* prerender_contents =
+ prerender_manager->GetPrerenderContents(tab);
+ if (prerender_contents) {
+ prerender_contents->Destroy(
+ prerender::FINAL_STATUS_SSL_CLIENT_CERTIFICATE_REQUESTED);
+ return;
+ }
+ }
+
GURL requesting_url("https://" + cert_request_info->host_and_port.ToString());
DCHECK(requesting_url.is_valid())
<< "Invalid URL string: https://"

Powered by Google App Engine
This is Rietveld 408576698