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

Unified Diff: chrome/browser/automation/automation_provider_observers.h

Issue 2813010: Revert 49862 - Measure loading time of several tabs.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 6 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/automation/automation_provider_observers.h
===================================================================
--- chrome/browser/automation/automation_provider_observers.h (revision 49869)
+++ chrome/browser/automation/automation_provider_observers.h (working copy)
@@ -36,18 +36,7 @@
const NotificationSource& source,
const NotificationDetails& details);
- // Caller owns the return value and is responsible for deleting it.
- // Example return value:
- // {'tabs': [{'start_time_ms': 1, 'stop_time_ms': 2.5},
- // {'start_time_ms': 0.5, 'stop_time_ms': 3}]}
- // stop_time_ms values may be null if WaitForInitialLoads has not finished.
- // Only includes entries for the |tab_count| tabs we are monitoring.
- // There is no defined ordering of the return value.
- DictionaryValue* GetTimingInformation() const;
-
private:
- class TabTime;
- typedef std::map<uintptr_t, TabTime> TabTimeMap;
typedef std::set<uintptr_t> TabSet;
void ConditionMet();
@@ -56,8 +45,7 @@
AutomationProvider* automation_;
size_t outstanding_tab_count_;
- base::TimeTicks init_time_;
- TabTimeMap loading_tabs_;
+ TabSet loading_tabs_;
TabSet finished_tabs_;
DISALLOW_COPY_AND_ASSIGN(InitialLoadObserver);
« no previous file with comments | « chrome/browser/automation/automation_provider.cc ('k') | chrome/browser/automation/automation_provider_observers.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698