Remove RenderViewHost parameter from WebContentsObserver::Did{Start|Stop}Loading.

extensions/browser/guest_view/guest_view_base.cc

 // 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.
 
 #include "extensions/browser/guest_view/guest_view_base.h"
 
 #include "base/lazy_instance.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/ui/zoom/page_zoom.h"
 #include "components/ui/zoom/zoom_controller.h"
@@ skipping 500 matching lines @@
   double zoom_factor = GetEmbedderZoomFactor();
   return lround(logical_pixels * zoom_factor);
 }
 
 double GuestViewBase::PhysicalPixelsToLogicalPixels(int physical_pixels) {
   DCHECK(physical_pixels >= 0);
   double zoom_factor = GetEmbedderZoomFactor();
   return physical_pixels / zoom_factor;
 }
 
-void GuestViewBase::DidStopLoading(content::RenderViewHost* render_view_host) {
+void GuestViewBase::DidStopLoading() {
+  content::RenderViewHost* rvh = web_contents()->GetRenderViewHost();
+
   if (IsPreferredSizeModeEnabled()) {

[Fady Samuel, 2015/03/14 04:25:02]

nit: Could you please get rid of these unnecessary braces while you're at it?
Thanks!

[Fabrice (no longer in Chrome), 2015/03/16 11:56:13]

Done.

-    render_view_host->EnablePreferredSizeMode();
+    rvh->EnablePreferredSizeMode();
   }
   if (!IsDragAndDropEnabled()) {
-    const char script[] = "window.addEventListener('dragstart', function() { "
-                          "  window.event.preventDefault(); "
-                          "});";
-    render_view_host->GetMainFrame()->ExecuteJavaScript(
-        base::ASCIIToUTF16(script));
+    const char script[] =
+        "window.addEventListener('dragstart', function() { "
+        "  window.event.preventDefault(); "
+        "});";
+    rvh->GetMainFrame()->ExecuteJavaScript(base::ASCIIToUTF16(script));
   }
-  DidStopLoading();
+  GuestViewDidStopLoading();
 }
 
 void GuestViewBase::RenderViewReady() {
   GuestReady();
 }
 
 void GuestViewBase::WebContentsDestroyed() {
   // Let the derived class know that its WebContents is in the process of
   // being destroyed. web_contents() is still valid at this point.
   // TODO(fsamuel): This allows for reentrant code into WebContents during
@@ skipping 276 matching lines @@
 void GuestViewBase::RegisterGuestViewTypes() {
   AppViewGuest::Register();
   ExtensionOptionsGuest::Register();
   ExtensionViewGuest::Register();
   MimeHandlerViewGuest::Register();
   SurfaceWorkerGuest::Register();
   WebViewGuest::Register();
 }
 
 }  // namespace extensions