We have a problem in the process on destroying WebContentsImpl because

chrome/browser/ui/passwords/manage_passwords_bubble_model.cc

 // Copyright 2013 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/passwords/manage_passwords_bubble_model.h"
 
 #include "chrome/browser/password_manager/password_store_factory.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/passwords/manage_passwords_bubble_ui_controller.h"
@@ skipping 111 matching lines @@
   password_manager::PasswordStore* password_store =
       PasswordStoreFactory::GetForProfile(profile, Profile::EXPLICIT_ACCESS)
           .get();
   DCHECK(password_store);
   if (action == REMOVE_PASSWORD)
     password_store->RemoveLogin(password_form);
   else
     password_store->AddLogin(password_form);
 }
 
-void ManagePasswordsBubbleModel::WebContentsDestroyed(
-    content::WebContents* web_contents) {
+void ManagePasswordsBubbleModel::WebContentsDestroyed() {
   // The WebContents have been destroyed.
   web_contents_ = NULL;

[jam, 2014/05/02 19:34:11]

please remove web_contents_ from this class and switch it to use web_contents()
instead?

[zverre, 2014/05/05 16:40:25]

Done.

 }