Index: content/public/test/web_contents_observer_sanity_checker.cc |
diff --git a/content/public/test/web_contents_observer_sanity_checker.cc b/content/public/test/web_contents_observer_sanity_checker.cc |
deleted file mode 100644 |
index 9b0fdccd344afcb497e998b8bdc517559d787996..0000000000000000000000000000000000000000 |
--- a/content/public/test/web_contents_observer_sanity_checker.cc |
+++ /dev/null |
@@ -1,257 +0,0 @@ |
-// 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 "content/public/test/web_contents_observer_sanity_checker.h" |
- |
-#include "base/strings/stringprintf.h" |
-#include "content/common/frame_messages.h" |
-#include "content/public/browser/render_frame_host.h" |
-#include "content/public/browser/render_process_host.h" |
-#include "content/public/browser/site_instance.h" |
-#include "content/public/browser/web_contents.h" |
-#include "content/public/browser/web_contents_observer.h" |
- |
-namespace content { |
- |
-namespace { |
- |
-const char kWebContentsObserverSanityCheckerKey[] = |
- "WebContentsObserverSanityChecker"; |
- |
-} // namespace |
- |
-// static |
-void WebContentsObserverSanityChecker::Enable(WebContents* web_contents) { |
- if (web_contents->GetUserData(&kWebContentsObserverSanityCheckerKey)) |
- return; |
- web_contents->SetUserData(&kWebContentsObserverSanityCheckerKey, |
- new WebContentsObserverSanityChecker(web_contents)); |
-} |
- |
-void WebContentsObserverSanityChecker::RenderFrameCreated( |
- RenderFrameHost* render_frame_host) { |
- CHECK(!web_contents_destroyed_); |
- std::pair<int, int> routing_pair = |
- std::make_pair(render_frame_host->GetProcess()->GetID(), |
- render_frame_host->GetRoutingID()); |
- bool frame_exists = !live_routes_.insert(routing_pair).second; |
- deleted_routes_.erase(routing_pair); |
- |
- if (frame_exists) { |
- CHECK(false) << "RenderFrameCreated called more than once for routing pair:" |
- << Format(render_frame_host); |
- } |
-} |
- |
-void WebContentsObserverSanityChecker::RenderFrameDeleted( |
- RenderFrameHost* render_frame_host) { |
- CHECK(!web_contents_destroyed_); |
- std::pair<int, int> routing_pair = |
- std::make_pair(render_frame_host->GetProcess()->GetID(), |
- render_frame_host->GetRoutingID()); |
- bool was_live = !!live_routes_.erase(routing_pair); |
- bool was_dead_already = !deleted_routes_.insert(routing_pair).second; |
- |
- if (was_dead_already) { |
- CHECK(false) << "RenderFrameDeleted called more than once for routing pair " |
- << Format(render_frame_host); |
- } else if (!was_live) { |
-// TODO(nick): Clients can easily ignore an unrecognized object, but it |
-// would be useful from a finding-bugs perspective if we could enable this |
-// check. |
-#if 0 |
- CHECK(false) << "RenderFrameDeleted called for routing pair " |
- << Format(render_frame_host) |
- << " for which RenderFrameCreated was never called"; |
-#endif |
- } |
-} |
- |
-void WebContentsObserverSanityChecker::RenderFrameForInterstitialPageCreated( |
- RenderFrameHost* render_frame_host) { |
- // TODO(nick): Record this. |
-} |
- |
-void WebContentsObserverSanityChecker::RenderFrameHostChanged( |
- RenderFrameHost* old_host, |
- RenderFrameHost* new_host) { |
- CHECK(new_host); |
- CHECK_NE(new_host, old_host); |
- |
- if (old_host) { |
- std::pair<int, int> routing_pair = |
- std::make_pair(old_host->GetProcess()->GetID(), |
- old_host->GetRoutingID()); |
- bool old_did_exist = !!current_hosts_.erase(routing_pair); |
- if (!old_did_exist) { |
- CHECK(false) |
- << "RenderFrameHostChanged called with old host that did not exist:" |
- << Format(old_host); |
- } |
- } |
- |
- std::pair<int, int> routing_pair = |
- std::make_pair(new_host->GetProcess()->GetID(), |
- new_host->GetRoutingID()); |
- bool host_exists = !current_hosts_.insert(routing_pair).second; |
- if (host_exists) { |
- CHECK(false) |
- << "RenderFrameHostChanged called more than once for routing pair:" |
- << Format(new_host); |
- } |
-} |
- |
-void WebContentsObserverSanityChecker::FrameDeleted( |
- RenderFrameHost* render_frame_host) { |
- // A frame can be deleted before RenderFrame in the renderer process is |
- // created, so there is not much that can be enforced here. |
- CHECK(!web_contents_destroyed_); |
-} |
- |
-void WebContentsObserverSanityChecker::DidStartProvisionalLoadForFrame( |
- RenderFrameHost* render_frame_host, |
- const GURL& validated_url, |
- bool is_error_page, |
- bool is_iframe_srcdoc) { |
- AssertRenderFrameExists(render_frame_host); |
-} |
- |
-void WebContentsObserverSanityChecker::DidCommitProvisionalLoadForFrame( |
- RenderFrameHost* render_frame_host, |
- const GURL& url, |
- ui::PageTransition transition_type) { |
- AssertRenderFrameExists(render_frame_host); |
-} |
- |
-void WebContentsObserverSanityChecker::DidFailProvisionalLoad( |
- RenderFrameHost* render_frame_host, |
- const GURL& validated_url, |
- int error_code, |
- const base::string16& error_description) { |
- AssertRenderFrameExists(render_frame_host); |
-} |
- |
-void WebContentsObserverSanityChecker::DidNavigateMainFrame( |
- const LoadCommittedDetails& details, |
- const FrameNavigateParams& params) { |
- AssertMainFrameExists(); |
-} |
- |
-void WebContentsObserverSanityChecker::DidNavigateAnyFrame( |
- RenderFrameHost* render_frame_host, |
- const LoadCommittedDetails& details, |
- const FrameNavigateParams& params) { |
- AssertRenderFrameExists(render_frame_host); |
-} |
- |
-void WebContentsObserverSanityChecker::DocumentAvailableInMainFrame() { |
- AssertMainFrameExists(); |
-} |
- |
-void WebContentsObserverSanityChecker::DocumentOnLoadCompletedInMainFrame() { |
- AssertMainFrameExists(); |
-} |
- |
-void WebContentsObserverSanityChecker::DocumentLoadedInFrame( |
- RenderFrameHost* render_frame_host) { |
- AssertRenderFrameExists(render_frame_host); |
-} |
- |
-void WebContentsObserverSanityChecker::DidFinishLoad( |
- RenderFrameHost* render_frame_host, |
- const GURL& validated_url) { |
- AssertRenderFrameExists(render_frame_host); |
-} |
- |
-void WebContentsObserverSanityChecker::DidFailLoad( |
- RenderFrameHost* render_frame_host, |
- const GURL& validated_url, |
- int error_code, |
- const base::string16& error_description) { |
- AssertRenderFrameExists(render_frame_host); |
-} |
- |
-void WebContentsObserverSanityChecker::DidGetRedirectForResourceRequest( |
- RenderFrameHost* render_frame_host, |
- const ResourceRedirectDetails& details) { |
- AssertRenderFrameExists(render_frame_host); |
-} |
- |
-void WebContentsObserverSanityChecker::DidOpenRequestedURL( |
- WebContents* new_contents, |
- RenderFrameHost* source_render_frame_host, |
- const GURL& url, |
- const Referrer& referrer, |
- WindowOpenDisposition disposition, |
- ui::PageTransition transition) { |
- AssertRenderFrameExists(source_render_frame_host); |
-} |
- |
-bool WebContentsObserverSanityChecker::OnMessageReceived( |
- const IPC::Message& message, |
- RenderFrameHost* render_frame_host) { |
- // TODO(nasko): FrameHostMsg_RenderProcessGone is delivered to |
- // WebContentsObserver since RenderFrameHost allows the delegate to handle |
- // the message first. This shouldn't happen, but for now handle it here. |
- // https://crbug.com/450799 |
- if (message.type() == FrameHostMsg_RenderProcessGone::ID) |
- return false; |
- |
-#if !defined(OS_MACOSX) |
-// TODO(avi): Disabled because of http://crbug.com/445054 |
- AssertRenderFrameExists(render_frame_host); |
-#endif |
- return false; |
-} |
- |
-void WebContentsObserverSanityChecker::WebContentsDestroyed() { |
- CHECK(!web_contents_destroyed_); |
- web_contents_destroyed_ = true; |
-} |
- |
-WebContentsObserverSanityChecker::WebContentsObserverSanityChecker( |
- WebContents* web_contents) |
- : WebContentsObserver(web_contents), web_contents_destroyed_(false) { |
- // Prime the pump with the initial objects. |
- // TODO(nasko): Investigate why this is needed. |
- RenderViewCreated(web_contents->GetRenderViewHost()); |
-} |
- |
-WebContentsObserverSanityChecker::~WebContentsObserverSanityChecker() { |
- CHECK(web_contents_destroyed_); |
-} |
- |
-void WebContentsObserverSanityChecker::AssertRenderFrameExists( |
- RenderFrameHost* render_frame_host) { |
- CHECK(!web_contents_destroyed_); |
- std::pair<int, int> routing_pair = |
- std::make_pair(render_frame_host->GetProcess()->GetID(), |
- render_frame_host->GetRoutingID()); |
- |
- bool render_frame_created_happened = live_routes_.count(routing_pair) != 0; |
- bool render_frame_deleted_happened = deleted_routes_.count(routing_pair) != 0; |
- |
- CHECK(render_frame_created_happened) |
- << "A RenderFrameHost pointer was passed to a WebContentsObserver " |
- << "method, but WebContentsObserver::RenderFrameCreated was never called " |
- << "for that RenderFrameHost: " << Format(render_frame_host); |
- CHECK(!render_frame_deleted_happened) |
- << "A RenderFrameHost pointer was passed to a WebContentsObserver " |
- << "method, but WebContentsObserver::RenderFrameDeleted had already been " |
- << "called on that frame:" << Format(render_frame_host); |
-} |
- |
-void WebContentsObserverSanityChecker::AssertMainFrameExists() { |
- AssertRenderFrameExists(web_contents()->GetMainFrame()); |
-} |
- |
-std::string WebContentsObserverSanityChecker::Format( |
- RenderFrameHost* render_frame_host) { |
- return base::StringPrintf( |
- "(%d, %d -> %s)", render_frame_host->GetProcess()->GetID(), |
- render_frame_host->GetRoutingID(), |
- render_frame_host->GetSiteInstance()->GetSiteURL().spec().c_str()); |
-} |
- |
-} // namespace content |