Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1941)

Unified Diff: chrome/browser/page_cycler/page_cycler.h

Issue 10386134: Record API fix adding FirstRun, and allowing repeat-count in capture (plus some browsertests along … (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Refined version with overlooked browsertest also fixed. Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/page_cycler/page_cycler.h
diff --git a/chrome/browser/page_cycler/page_cycler.h b/chrome/browser/page_cycler/page_cycler.h
new file mode 100644
index 0000000000000000000000000000000000000000..daa02e1f9b24d157618587d0d68bff73f05aa59a
--- /dev/null
+++ b/chrome/browser/page_cycler/page_cycler.h
@@ -0,0 +1,149 @@
+// 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 CHROME_BROWSER_PAGE_CYCLER_PAGE_CYCLER_H_
+#define CHROME_BROWSER_PAGE_CYCLER_PAGE_CYCLER_H_
+#pragma once
+
+#include "base/memory/ref_counted.h"
+#include "base/file_path.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_list.h"
+#include "content/public/browser/navigation_controller.h"
+#include "content/public/browser/notification_observer.h"
+#include "content/public/browser/notification_registrar.h"
+#include "content/public/browser/web_contents_observer.h"
+
+namespace base {
+class TimeTicks;
+} // namespace base
+
+// Performance test to track the resources used and speed with which chromium
+// fully loads a given set of URLs. This class is created on the UI thread and
+// does most of its work there. However, some work happens on background threads
+// too; those are named with 'OnBackgroundThread'.
+class PageCycler : public base::RefCountedThreadSafe<PageCycler>,
+ public BrowserList::Observer,
+ public content::WebContentsObserver {
+ public:
+ PageCycler(Browser* browser, FilePath urls_file, FilePath errors_file);
+
+ // Begin running the page cycler.
+ void Run(const int& total_iterations);
+
+ // content::WebContentsObserver
+ virtual void DidFinishLoad(int64 frame_id,
+ const GURL& validated_url,
+ bool is_main_frame) OVERRIDE;
+ virtual void DidFailProvisionalLoad(
+ int64 frame_id,
+ bool is_main_frame,
+ const GURL& validated_url,
+ int error_code,
+ const string16& error_description) OVERRIDE;
+
+ // This method should never be necessary while running PageCycler; this is
+ // for testing purposes only.
+ const std::vector<GURL>* urls_for_test() { return &urls_; }
+
+ // Set the optional stats file; if this is not set prior to calling Run(),
+ // then no stats are recorded.
+ void set_stats_file(const FilePath& stats_file) { stats_file_ = stats_file; }
+
+ protected:
+ virtual ~PageCycler();
+
+ private:
+ friend class base::RefCountedThreadSafe<PageCycler>;
+ friend class MockPageCycler;
+
+ // Check to see if a load callback is valid; i.e. the load should be from the
+ // main frame, the url should not be a chrome error url, and |url_index|
+ // should not be 0.
+ bool IsLoadCallbackValid(const GURL& validated_url,
+ bool is_main_frame);
+
+ // Read in the urls from |urls_file_| and store them in |urls_|.
+ void ReadURLsOnBackgroundThread();
+
+ // Perform any initial setup neccessary, and begin visiting the pages.
+ void BeginCycle();
+
+ // If |url_index_| points to a valid position in |urls_|, load the url,
+ // capturing any statistics information. Otherwise, call WriteResults.
+ void LoadNextURL();
+
+ // Complete statistics gathering for the finished visit, and try to load the
+ // next url.
+ void LoadSucceeded();
+
+ // Inidicate that the load failed with an error; try to load the next url.
+ void LoadFailed(const GURL& url, const string16& error_description);
+
+ // Finalize the output strings.
+ void PrepareResults();
+
+ // Write the data stored within output to the file indicated by |stats_file_|,
+ // if |stats_file_| is not empty. Write any errors to |errors_file_|.
+ void WriteResultsOnBackgroundThread(std::string output);
+
+ // Perform any necessary cleanup and exit |browser_|.
+ void Finish();
+
+ // Called when the Browser to which |browser_| points is closed; exits
+ // PageCycler.
+ void Abort();
+
+ // BrowserList::Observer
+ virtual void OnBrowserAdded(const Browser* browser) OVERRIDE;
+ virtual void OnBrowserRemoved(const Browser* browser) OVERRIDE;
+
+ // The Browser context in which the page cycler is running.
+ Browser* browser_;
+
+ // The path to the file containing the list of urls to visit.
+ FilePath urls_file_;
+
+ // The path to the file to which we write any errors encountered.
+ FilePath errors_file_;
+
+ // The path to the file to which we write the statistics (optional, may be
+ // an empty path).
+ FilePath stats_file_;
+
+ // The list of urls to visit.
+ std::vector<GURL> urls_;
+
+ // The current index into the |urls_| vector.
+ size_t url_index_;
+
+ // The number of total iterations to be run.
+ int total_iterations_;
+
+ // The number of the current iteration.
+ int current_iteration_;
+
+ // The generated string of urls which we have visited; this is built one url
+ // at a time as we iterate through the |urls_| vector. This is primarily
+ // included for interfacing with the previous page_cycler's output style.
+ std::string urls_string_;
+
+ // The generated string of the times taken to visit each url. As with
+ // |urls_string_|, this is built as we visit each url, and is primarily to
+ // produce output similar to the previous page_cycler's.
+ std::string timings_string_;
+
+ // The time at which we begin the process of loading the next url; this is
+ // used to calculate the time taken for each url load.
+ base::TimeTicks initial_time_;
+
+ // Indicates the abort status of the page cycler; true means aborted.
+ bool aborted_;
+
+ string16 error_;
+
+ DISALLOW_COPY_AND_ASSIGN(PageCycler);
+};
+
+#endif // CHROME_BROWSER_PAGE_CYCLER_PAGE_CYCLER_H_

Powered by Google App Engine
This is Rietveld 408576698