Fix RenderFrameCreated and RenderFrameDeleted WebContentsObserver methods

content/public/browser/web_contents.h

Index: content/public/browser/web_contents.h
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index 879e8d8513c03fa6ccb621249098bb66873a0fab..65e6f2317c98fea9fbc273615e1b392e4af6c345 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -122,6 +122,9 @@ class WebContents : public PageNavigator,
     // Used to specify the location context which display the new view should
     // belong. This can be nullptr if not needed.
     gfx::NativeView context;
+
+    // Created by the renderer process.
+    bool renderer_created;

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

Hmm, this is kind of difficult to understand without looking at the
implementation.  CreateParams are instructions to the WebContents constructor,
so let's clarify when callers should set this to true.

As is, it might mean any of the following (or something else):
 - This WC will definitely be put into an existing renderer process.
 - A renderer process asked for this WC to be created, but it could end up in a
new/different process.
 - A renderer process asked for this WC to be created, and thus its main
RenderFrame has already been created (?).

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

It is the last one of them. I've renamed the parameter and added a better
comment.

   };
 
   // Creates a new WebContents.