Fix RenderFrameCreated and RenderFrameDeleted WebContentsObserver methods

content/public/browser/web_contents.h

 // Copyright (c) 2012 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 CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 
 #include <set>
 
 #include "base/basictypes.h"
@@ skipping 104 matching lines @@
 
     // True if the contents should be initially hidden.
     bool initially_hidden;
 
     // If non-null then this WebContents will be hosted by a BrowserPlugin.
     BrowserPluginGuestDelegate* guest_delegate;
 
     // Used to specify the location context which display the new view should
     // belong. This can be nullptr if not needed.
     gfx::NativeView context;
+
+    // Used to specify that the new WebContents creation is driven by the
+    // renderer. In this case, the renderer-side objects, such as

[Charlie Reis, 2015/02/12 23:51:26]

nit: Let's try to say "renderer process" when we can, to avoid ambiguity about
process vs RenderView vs RenderFrame.

[nasko, 2015/02/13 00:18:52]

Good catch! I've been trying to avoid using just renderer, so fixed!

+    // RenderFrameHost, have already been created on the renderer side, and

[Charlie Reis, 2015/02/12 23:51:26]

RenderFrame  :)

[nasko, 2015/02/13 00:18:51]

Done.

+    // WebContents construction should take this into account.
+    bool renderer_initiated_creation;
   };
 
   // Creates a new WebContents.
   CONTENT_EXPORT static WebContents* Create(const CreateParams& params);
 
   // Similar to Create() above but should be used when you need to prepopulate
   // the SessionStorageNamespaceMap of the WebContents. This can happen if
   // you duplicate a WebContents, try to reconstitute it from a saved state,
   // or when you create a new WebContents based on another one (eg., when
   // servicing a window.open() call).
@@ skipping 483 matching lines @@
 
  private:
   // This interface should only be implemented inside content.
   friend class WebContentsImpl;
   WebContents() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_