Fix RenderFrameCreated and RenderFrameDeleted WebContentsObserver methods

content/public/test/web_contents_observer_sanity_checker.h

Index: content/public/test/web_contents_observer_sanity_checker.h
diff --git a/content/public/test/web_contents_observer_sanity_checker.h b/content/public/test/web_contents_observer_sanity_checker.h
index 2d702d0859e34184f694cf7fd666279e6be5fc3d..6fe06c435a52134ae33fa7a8a51b6edced0faf42 100644
--- a/content/public/test/web_contents_observer_sanity_checker.h
+++ b/content/public/test/web_contents_observer_sanity_checker.h
@@ -84,11 +84,16 @@ class WebContentsObserverSanityChecker : public WebContentsObserver,
   ~WebContentsObserverSanityChecker() override;
 
   std::string Format(RenderFrameHost* render_frame_host);
-  void AssertFrameExists(RenderFrameHost* render_frame_host);
+  void AssertRenderFrameExists(RenderFrameHost* render_frame_host);
   void AssertMainFrameExists();
 
+  std::set<std::pair<int, int>> current_hosts_;
+  std::set<std::pair<int, int>> deleted_hosts_;
   std::set<std::pair<int, int>> live_routes_;
   std::set<std::pair<int, int>> deleted_routes_;
+  base::hash_map<std::pair<int, int>, std::string> render_frame_created_stacks_;

[Charlie Reis, 2015/02/12 00:29:31]

I think we should leave this out of this CL.  If we want to add it for
debug-only logging, let's do that in a follow-up CL so that it's easier to read
in git blame.

[nasko, 2015/02/12 17:52:26]

Done.

+  base::hash_map<std::pair<int, int>, std::string> render_frame_deleted_stacks_;
+
   bool web_contents_destroyed_;
 
   DISALLOW_COPY_AND_ASSIGN(WebContentsObserverSanityChecker);