Add WebContents accessor in ContentViewCore interface.

content/browser/android/content_view_core_impl.h

Index: content/browser/android/content_view_core_impl.h
diff --git a/content/browser/android/content_view_core_impl.h b/content/browser/android/content_view_core_impl.h
index adc40f3bb9586fda077c6ce938c49e0428e27845..32fcb28391b64e044ab089777c41049abc6b71bc 100644
--- a/content/browser/android/content_view_core_impl.h
+++ b/content/browser/android/content_view_core_impl.h
@@ -169,8 +169,7 @@ class ContentViewCoreImpl : public ContentViewCore,
 
   gfx::Rect GetBounds() const;
 
-  WebContents* web_contents() const { return web_contents_; }
-
+  virtual WebContents* web_contents() const OVERRIDE;

[Ted C, 2012/09/17 18:19:44]

Hmm...I thought the hacker style naming was only for simple inlined getters.  In
this case, does it need to become GetWebContents?

[jam, 2012/09/18 19:15:22]

correct, this should be GetWebContents()

web_contents() in content/public is on WebContentsObserver which isn't a virtual
method, but is a convenience getter. I see there's also one on
RenderViewHostTestHarness, but that's wrong and was supposed to be fixed but
wasn't yet.

[nilesh, 2012/09/18 20:47:12]

Renamed to GetWebContents()

   virtual void LoadUrl(NavigationController::LoadURLParams& params) OVERRIDE;
 
  private: