Fix cleanup in CredentialManagerClientTest

components/password_manager/content/renderer/credential_manager_client_browsertest.cc

 // Copyright 2014 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 "components/password_manager/content/common/credential_manager_messages.h"
 #include "components/password_manager/content/renderer/credential_manager_client.h"
 #include "content/public/test/render_view_test.h"
 #include "ipc/ipc_test_sink.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/WebKit/public/platform/WebCredential.h"
@@ skipping 21 matching lines @@
   ~MAYBE_CredentialManagerClientTest() override {}
 
   void SetUp() override {
     content::RenderViewTest::SetUp();
     credential_.reset(new blink::WebLocalCredential("", "", GURL(), ""));
     client_.reset(new CredentialManagerClient(view_));
   }
 
   void TearDown() override {
     credential_.reset();
+    client_.reset();
     content::RenderViewTest::TearDown();
   }
 
   IPC::TestSink& sink() { return render_thread_->sink(); }
 
   blink::WebCredential* credential() { return credential_.get(); }
 
   // The browser's response to any of the messages the client sends must contain
   // a request ID so that the client knows which request is being serviced. This
   // method grabs the ID from an outgoing |message_id| message, and sets the
@@ skipping 184 matching lines @@
   EXPECT_TRUE(ExtractRequestId(CredentialManagerHostMsg_RequestCredential::ID,
                                request_id));
 
   CredentialInfo info;  // Send an empty credential in response.
   client_->OnSendCredential(request_id, info);
   EXPECT_TRUE(callback_succeeded());
   EXPECT_FALSE(callback_errored());
 }
 
 }  // namespace password_manager