Clean up a few TODO items.

content/test/web_contents_tester.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_TEST_WEB_CONTENTS_TESTER_H_
 #define CONTENT_TEST_WEB_CONTENTS_TESTER_H_
 #pragma once
 
 #include "content/public/common/page_transition_types.h"
 
@@ skipping 66 matching lines @@
   // Simulates the appropriate RenderView (pending if any, current otherwise)
   // sending a navigate notification for the NavigationController pending entry.
   virtual void CommitPendingNavigation() = 0;
 
   // Only implementations retrieved via the deprecated
   // CreateTestWebContentsFor... methods above will implement this
   // method.  It retrieves the number of times the focus-related calls
   // in question have been made.
   virtual int GetNumberOfFocusCalls() = 0;
 
-  // TODO(joi): Rename this and other hacker_style virtual methods.
-  virtual content::RenderViewHost* pending_rvh() const = 0;
+  virtual content::RenderViewHost* GetPendingRvh() const = 0;

[jam, 2012/03/22 17:15:41]

nit: since RVH is an acronym it should be GetPendingRVH. or even better, just
GetPendingRenderViewHost to match the rest of the code which usually doesn't
shorten it (i.e. this method calls out to pending_render_view_host() I believe).

[Jói, 2012/03/22 17:39:58]

Done.

 
   // Creates a pending navigation to the given URL with the default parameters
   // and then commits the load with a page ID one larger than any seen. This
   // emulates what happens on a new navigation.
   virtual void NavigateAndCommit(const GURL& url) = 0;
 
   // Simulates the current RVH notifying that it has unloaded so that the
   // pending RVH navigation can proceed.
   // Does nothing if no cross-navigation is pending.
   virtual void ProceedWithCrossSiteNavigation() = 0;
@@ skipping 10 matching lines @@
       const content::Referrer& referrer,
       content::PageTransition transition) = 0;
 
   // Promote GetWebkitPrefs to public.
   virtual WebPreferences TestGetWebkitPrefs() = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_TEST_WEB_CONTENTS_TESTER_H_