This patch improves the way that GuestViewManager tracks the destruction of GuestView embedders.

components/guest_view/browser/guest_view_manager.h

 // 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.
 
 #ifndef COMPONENTS_GUEST_VIEW_BROWSER_GUEST_VIEW_MANAGER_H_
 #define COMPONENTS_GUEST_VIEW_BROWSER_GUEST_VIEW_MANAGER_H_
 
 #include <map>
 #include <vector>
 
@@ skipping 114 matching lines @@
   bool ForEachGuest(content::WebContents* owner_web_contents,
                     const GuestCallback& callback) override;
   content::WebContents* GetFullPageGuest(
       content::WebContents* embedder_web_contents) override;
 
  protected:
   friend class GuestViewBase;
   friend class GuestViewEvent;
   friend class GuestViewMessageFilter;
 
+  class EmbedderRenderProcessHostObserver;
+
   // These methods are virtual so that they can be overriden in tests.
 
   virtual void AddGuest(int guest_instance_id,
                         content::WebContents* guest_web_contents);
   virtual void RemoveGuest(int guest_instance_id);
 
   // This method is called when the embedder process with ID
-  // |embedder_process_id| is about to be destroyed.
-  virtual void EmbedderWillBeDestroyed(int embedder_process_id);
+  // |embedder_process_id| has been destroyed.
+  virtual void EmbedderProcessDestroyed(int embedder_process_id);
 
   // Called when a GuestView has been created in JavaScript.
   virtual void ViewCreated(int embedder_process_id,
                            int view_instance_id,
                            const std::string& view_type);
 
   // Called when a GuestView has been garbage collected in JavaScript.
   virtual void ViewGarbageCollected(int embedder_process_id,
                                     int view_instance_id);
 
+  // Calls all destruction callbacks registered for the GuestView identified by
+  // |embedder_process_id| and |view_instance_id|.
+  void CallViewDestructionCallbacks(int embedder_process_id,
+                                    int view_instance_id);
+
+  // Calls all destruction callbacks registered for GuestViews in the embedder
+  // with ID |embedder_process_id|.
+  void CallViewDestructionCallbacks(int embedder_process_id);
+
   // Creates a guest of the provided |view_type|.
   GuestViewBase* CreateGuestInternal(content::WebContents* owner_web_contents,
                                      const std::string& view_type);
 
   // Adds GuestView types to the GuestView registry.
   void RegisterGuestViewTypes();
 
   // Indicates whether the provided |guest| can be used in the context it has
   // been created.
   bool IsGuestAvailableToContext(GuestViewBase* guest);
@@ skipping 44 matching lines @@
 
   using GuestInstanceIDMap = std::map<ElementInstanceKey, int>;
   GuestInstanceIDMap instance_id_map_;
 
   // The reverse map of GuestInstanceIDMap.
   using GuestInstanceIDReverseMap = std::map<int, ElementInstanceKey>;
   GuestInstanceIDReverseMap reverse_instance_id_map_;
 
   using GuestViewCreateFunction =
       base::Callback<GuestViewBase*(content::WebContents*)>;
-  using GuestViewCleanUpFunction = base::Callback<void(int, int)>;
+  using GuestViewCleanUpFunction =
+      base::Callback<void(content::BrowserContext*, int, int)>;
   struct GuestViewData {
     GuestViewData(const GuestViewCreateFunction& create_function,
                   const GuestViewCleanUpFunction& cleanup_function);
     ~GuestViewData();
     const GuestViewCreateFunction create_function;
     const GuestViewCleanUpFunction cleanup_function;
   };
   using GuestViewMethodMap = std::map<std::string, GuestViewData>;
   GuestViewMethodMap guest_view_registry_;
 
@@ skipping 12 matching lines @@
 
   scoped_ptr<GuestViewManagerDelegate> delegate_;
 
   // |view_destruction_callback_map_| maps from embedder process ID to view ID
   // to a vector of callback functions to be called when that view is destroyed.
   using Callbacks = std::vector<base::Closure> ;
   using CallbacksForEachViewID = std::map<int, Callbacks> ;
   using CallbacksForEachEmbedderID = std::map<int, CallbacksForEachViewID> ;
   CallbacksForEachEmbedderID view_destruction_callback_map_;
 
+  ScopedVector<EmbedderRenderProcessHostObserver>
+      embedder_render_process_host_observers_;
+
   DISALLOW_COPY_AND_ASSIGN(GuestViewManager);
 };
 
 }  // namespace guest_view
 
 #endif  // COMPONETS_GUEST_VIEW_BROWSER_GUEST_VIEW_MANAGER_H_