GuestViewContainer lifetime should only be managed by BrowserPlugin and not RenderFrameObserver

extensions/renderer/guest_view/guest_view_container.cc

Index: extensions/renderer/guest_view/guest_view_container.cc
diff --git a/extensions/renderer/guest_view/guest_view_container.cc b/extensions/renderer/guest_view/guest_view_container.cc
index 05ccf68bbe376603f3e6da4da2f67a6bf688da2e..40051b103c17c11d8fc4aa180bdecc6448709a3c 100644
--- a/extensions/renderer/guest_view/guest_view_container.cc
+++ b/extensions/renderer/guest_view/guest_view_container.cc
@@ -23,8 +23,7 @@ GuestViewContainer::GuestViewContainer(
 GuestViewContainer::~GuestViewContainer() {
 }
 
-void GuestViewContainer::SetElementInstanceID(
-    int element_instance_id) {
+void GuestViewContainer::SetElementInstanceID(int element_instance_id) {
   element_instance_id_ = element_instance_id;
 }
 
@@ -42,6 +41,11 @@ void GuestViewContainer::DidReceiveData(const char* data, int data_length) {
   html_string_ += value;
 }
 
+void GuestViewContainer::OnDestruct() {
+  // GuestViewContainer's lifetime is managed by BrowserPlugin so don't let
+  // RenderFrameObserver self-destruct here.
+}
+
 bool GuestViewContainer::OnMessageReceived(const IPC::Message& message) {
   if (message.type() != ExtensionMsg_CreateMimeHandlerViewGuestACK::ID)
     return false;