Per the Chromium style guide/dos-and-donts doc, don't inline non-"cheap

chrome/browser/password_manager/password_manager_browsertest.cc

 // Copyright 2013 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 <string>
 
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/infobars/confirm_infobar_delegate.h"
 #include "chrome/browser/infobars/infobar_service.h"
 #include "chrome/browser/password_manager/password_store_factory.h"
@@ skipping 23 matching lines @@
         message_loop_runner_(new content::MessageLoopRunner),
         info_bar_shown_(false),
         infobar_service_(InfoBarService::FromWebContents(web_contents)) {
     registrar_.Add(this,
                    chrome::NOTIFICATION_TAB_CONTENTS_INFOBAR_ADDED,
                    content::Source<InfoBarService>(infobar_service_));
   }
 
   virtual ~NavigationObserver() {}
 
-  void Wait() {
-    message_loop_runner_->Run();
-  }
-
-  bool InfoBarWasShown() {
-    return info_bar_shown_;
-  }
-
   // content::NotificationObserver:
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE {
     // Accept in the infobar.
     InfoBarDelegate* infobar = infobar_service_->infobar_at(0);
     ConfirmInfoBarDelegate* confirm_infobar =
         infobar->AsConfirmInfoBarDelegate();
     confirm_infobar->Accept();
     info_bar_shown_ = true;
   }
 
-  // content::WebContentsObserver
+  // content::WebContentsObserver:
   virtual void DidFinishLoad(
       int64 frame_id,
       const GURL& validated_url,
       bool is_main_frame,
       content::RenderViewHost* render_view_host) OVERRIDE {
     message_loop_runner_->Quit();
   }
 
+  bool infobar_shown() { return info_bar_shown_; }
+
+  void Wait() {
+    message_loop_runner_->Run();
+  }
+
  private:
   scoped_refptr<content::MessageLoopRunner> message_loop_runner_;
   bool info_bar_shown_;
   content::NotificationRegistrar registrar_;
   InfoBarService* infobar_service_;
 };
 
 }  // namespace
 
 class PasswordManagerBrowserTest : public InProcessBrowserTest {
@@ skipping 28 matching lines @@
   // in its onsubmit handler but instead logs in/navigates via XHR.
   // Note that calling 'submit()' on a form with javascript doesn't call
   // the onsubmit handler, so we click the submit button instead.
   NavigationObserver observer(WebContents());
   std::string fill_and_submit =
       "document.getElementById('username_field').value = 'temp';"
       "document.getElementById('password_field').value = 'random';"
       "document.getElementById('submit_button').click()";
   ASSERT_TRUE(content::ExecuteScript(RenderViewHost(), fill_and_submit));
   observer.Wait();
-  EXPECT_TRUE(observer.InfoBarWasShown());
+  EXPECT_TRUE(observer.infobar_shown());
 }
 
 IN_PROC_BROWSER_TEST_F(PasswordManagerBrowserTest, NoPromptForOtherXHR) {
   ASSERT_TRUE(test_server()->Start());
 
   GURL url = test_server()->GetURL("files/password/password_xhr_submit.html");
   ui_test_utils::NavigateToURL(browser(), url);
 
   // Verify that if random XHR navigation occurs, we don't try and save the
   // password.
   //
   // We may want to change this functionality in the future to account for
   // cases where the element that users click on isn't a submit button.
   NavigationObserver observer(WebContents());
   std::string fill_and_navigate =
       "document.getElementById('username_field').value = 'temp';"
       "document.getElementById('password_field').value = 'random';"
       "send_xhr()";
   ASSERT_TRUE(content::ExecuteScript(RenderViewHost(), fill_and_navigate));
   observer.Wait();
-  EXPECT_FALSE(observer.InfoBarWasShown());
+  EXPECT_FALSE(observer.infobar_shown());
 }