Record API fix adding FirstRun, and allowing repeat-count in capture (plus some browsertests along …

chrome/browser/chrome_browser_main.h

Index: chrome/browser/chrome_browser_main.h
diff --git a/chrome/browser/chrome_browser_main.h b/chrome/browser/chrome_browser_main.h
index 71afa032b3a9a608fe0786427b1afa1e240ad531..d078254a45b8798b7a659184424bded20d754d4a 100644
--- a/chrome/browser/chrome_browser_main.h
+++ b/chrome/browser/chrome_browser_main.h
@@ -72,12 +72,15 @@ class ChromeBrowserMainParts : public content::BrowserMainParts {
   virtual void PostDestroyThreads() OVERRIDE;
 
   // Additional stages for ChromeBrowserMainExtraParts. These stages are called
-  // in order from PreMainMessageLoopStart(). See implementation for details.
+  // in order from PreMainMessageLoopRun(). See implementation for details.
   virtual void PreProfileInit();
   virtual void PostProfileInit();
   virtual void PreBrowserStart();
   virtual void PostBrowserStart();
 
+  // Runs the PageCycler; called if the switch kVisitURLs is present.
+  virtual void RunPageCycler();

[Aaron Boodman, 2012/05/16 23:37:00]

Does this need to be virtual?

[clintstaley, 2012/05/31 01:54:30]

I usually make large methods virtual since the polymorphic overhead is minimal
in C++, and it's more flexible that way.  Is this not wanted, tho?

+
   // Displays a warning message that we can't find any locale data files.
   virtual void ShowMissingLocaleMessageBox() = 0;