C++ readability review for lazyboy@

content/browser/browser_plugin/browser_plugin_host_browsertest.cc

 // Copyright 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.
 
 #include "base/command_line.h"
 #include "base/memory/singleton.h"
 #include "base/run_loop.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 127 matching lines @@
 
   DISALLOW_COPY_AND_ASSIGN(TestBrowserPluginHostFactory);
 };
 
 // Test factory class for browser plugin that creates guests with short hang
 // timeout.
 class TestShortHangTimeoutGuestFactory : public TestBrowserPluginHostFactory {
  public:
   virtual BrowserPluginGuest* CreateBrowserPluginGuest(
       int instance_id, WebContentsImpl* web_contents) OVERRIDE {
-    BrowserPluginGuest* guest =
+    TestBrowserPluginGuest* guest =
         new TestBrowserPluginGuest(instance_id, web_contents);
-    guest->set_guest_hang_timeout_for_testing(TestTimeouts::tiny_timeout());
+    guest->set_guest_hang_timeout(TestTimeouts::tiny_timeout());
     return guest;
   }
 
   // Singleton getter.
   static TestShortHangTimeoutGuestFactory* GetInstance() {
     return Singleton<TestShortHangTimeoutGuestFactory>::get();
   }
 
  protected:
   TestShortHangTimeoutGuestFactory() {}
@@ skipping 642 matching lines @@
 
   // Attempt a navigation to chrome-guest://abc123, which is a valid URL. But it
   // should be blocked because the scheme isn't web-safe or a pseudo-scheme.
   ExecuteSyncJSFunction(
       test_embedder()->web_contents()->GetRenderViewHost(),
       base::StringPrintf("SetSrc('%s://abc123');", kGuestScheme));
   EXPECT_TRUE(delegate->load_aborted());
   EXPECT_TRUE(delegate->load_aborted_url().is_valid());
 }
 
+

[Charlie Reis, 2013/11/13 19:04:28]

nit: I don't think we want to add the extra line here.

[lazyboy, 2013/11/13 19:21:11]

Right, Done.

 }  // namespace content