Remove calls of PrerenderTracker::TryCancel and TryCancelOnIOThread in ChromeResourceDispatcherHost…

chrome/browser/ui/login/login_prompt.cc

Index: chrome/browser/ui/login/login_prompt.cc
===================================================================
--- chrome/browser/ui/login/login_prompt.cc	(revision 243099)
+++ chrome/browser/ui/login/login_prompt.cc	(working copy)
@@ -12,6 +12,7 @@
 #include "base/synchronization/lock.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/password_manager/password_manager.h"
+#include "chrome/browser/prerender/prerender_contents.h"
 #include "chrome/browser/tab_contents/tab_util.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_registrar.h"
@@ -427,6 +428,13 @@
     return;
   }
 
+  prerender::PrerenderContents* prerender_contents =
+      prerender::PrerenderContents::FromWebContents(parent_contents);
+  if (prerender_contents) {
+    prerender_contents->Destroy(prerender::FINAL_STATUS_AUTH_NEEDED);

[davidben, 2014/01/06 22:14:47]

Do we want to call handler->CancelAuth() here too? Not sure it matters much
since the WebContents is going away, but that's what the old codepath did. Dunno
if some other component is sensitive to the notification.

[jam, 2014/01/06 22:21:00]

this will be equivalent to closing the tab; and the code must handle that code
path. the tests all pass, so I'm inclined to leave as is and not add extra code
unless I know for sure that it's needed

+    return;
+  }
+
   PasswordManager* password_manager =
       PasswordManager::FromWebContents(parent_contents);
   if (!password_manager) {