Crash fix: Remove a NOTREACHED that is wrong.

chrome/browser/ui/login/login_prompt.cc

 // Copyright (c) 2011 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/login/login_prompt.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 431 matching lines @@
     // not hosted by a tab (e.g. an extension). Cancel just in case
     // (cancelling twice is a no-op).
     handler->CancelAuth();
     return;
   }
 
   // Tell the password manager to look for saved passwords.
   TabContentsWrapper* wrapper =
       TabContentsWrapper::GetCurrentWrapperForContents(parent_contents);
   if (!wrapper)
-    NOTREACHED() << "Login dialog created for TabContents with no wrapper";
+    return;

[mattm, 2011/11/30 23:15:28]

So if we think this is an expected behavior, should it also have a CancelAuth
like above?

[James Hawkins, 2011/12/01 21:37:15]

Done.

 
   PasswordManager* password_manager = wrapper->password_manager();
   std::vector<PasswordForm> v;
   MakeInputForPasswordManager(request_url, auth_info, handler, &v);
   password_manager->OnPasswordFormsFound(v);
   handler->SetPasswordManager(password_manager);
 
   // The realm is controlled by the remote server, so there is no reason
   // to believe it is of a reasonable length.
   string16 elided_realm;
@@ skipping 14 matching lines @@
 
 LoginHandler* CreateLoginPrompt(net::AuthChallengeInfo* auth_info,
                                 net::URLRequest* request) {
   LoginHandler* handler = LoginHandler::Create(auth_info, request);
   BrowserThread::PostTask(
       BrowserThread::UI, FROM_HERE,
       base::Bind(&LoginDialogCallback, request->url(),
                  make_scoped_refptr(auth_info), make_scoped_refptr(handler)));
   return handler;
 }