Allow severing connection between dispatcher and injector

content/browser/intents/intent_injector_unittest.cc

 // Copyright (c) 2012 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 "base/bind.h"
+#include "base/memory/weak_ptr.h"
 #include "base/utf_string_conversions.h"
 #include "content/browser/intents/intent_injector.h"
 #include "content/browser/intents/internal_web_intents_dispatcher.h"
 #include "content/browser/renderer_host/test_render_view_host.h"
 #include "content/browser/web_contents/test_web_contents.h"
 #include "content/common/intents_messages.h"
 #include "content/common/view_messages.h"
 #include "content/test/web_contents_tester.h"
 #include "content/test/mock_render_process_host.h"
 #include "ipc/ipc_channel.h"
@@ skipping 65 matching lines @@
                        std::string());
   ASSERT_TRUE(contents()->cross_navigation_pending());
   ASSERT_TRUE(pending_rvh());
   injector->RenderViewCreated(pending_test_rvh());
 
   // The injector is still operating on the WebContents with the ddd.com
   // initial page, so no intent data is sent to this new renderer.
   EXPECT_TRUE(NULL == process()->sink().GetFirstMessageMatching(
       IntentsMsg_SetWebIntentData::ID));
 }
+
+TEST_F(IntentInjectorTest, AbandonDeletes) {
+  IntentInjector* injector = new IntentInjector(web_contents());
+
+  base::WeakPtr<IntentInjector> weak_injector =
+      injector->weak_factory_.GetWeakPtr();
+  EXPECT_FALSE(weak_injector.get() == NULL);
+  weak_injector->Abandon();
+  EXPECT_TRUE(weak_injector.get() == NULL);
+
+  // Clean up leftovers from the framework.
+  delete dispatcher_;

[Greg Billock, 2012/04/25 14:26:52]

Instead, call the dispatcher->SendReplyMessage. That'll test explicitly that it
won't crash and also delete it.

[groby-ooo-7-16, 2012/04/25 16:40:10]

Done.

+
+}