diff options
| author | tony@chromium.org <tony@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-02 17:18:46 +0000 |
|---|---|---|
| committer | tony@chromium.org <tony@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-02 17:18:46 +0000 |
| commit | c81202a0eff6e02942740f92944265e32f411376 (patch) | |
| tree | 4c7066278167b14d6cddd82e1335e6ca3cd46774 /chrome/browser/dom_ui/new_tab_ui.h | |
| parent | c28d7d66e345f8e87660200231998daaf4653622 (diff) | |
| download | chromium_src-c81202a0eff6e02942740f92944265e32f411376.zip chromium_src-c81202a0eff6e02942740f92944265e32f411376.tar.gz chromium_src-c81202a0eff6e02942740f92944265e32f411376.tar.bz2 | |
Fix the NewTabUI load UMA metric.
We were attaching the paint observer to the a RenderWidgetHost that
was getting immediately swapped out when the new tab page started
to load. Instead, wait and use the DOM UI callback methods which
happen right after NewTabUI is created.
Review URL: http://codereview.chromium.org/242115
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27850 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/dom_ui/new_tab_ui.h')
| -rw-r--r-- | chrome/browser/dom_ui/new_tab_ui.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/chrome/browser/dom_ui/new_tab_ui.h b/chrome/browser/dom_ui/new_tab_ui.h index bc60804..e5d1094 100644 --- a/chrome/browser/dom_ui/new_tab_ui.h +++ b/chrome/browser/dom_ui/new_tab_ui.h @@ -23,6 +23,11 @@ class NewTabUI : public DOMUI, explicit NewTabUI(TabContents* manager); ~NewTabUI(); + // Override DOMUI methods so we can hook up the paint timer to the render + // view host. + virtual void RenderViewCreated(RenderViewHost* render_view_host); + virtual void RenderViewReused(RenderViewHost* render_view_host); + static void RegisterUserPrefs(PrefService* prefs); // Whether we should use the old new tab page. |