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

Side by Side Diff: content/browser/web_contents/web_contents_impl.h

Issue 10945042: Add UMA to measure potential impact of Fast Tab Close. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Forward the "preceed time" from render view host manager. Created 8 years, 2 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 5 #ifndef CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
6 #define CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 6 #define CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 10
(...skipping 408 matching lines...) Expand 10 before | Expand all | Expand 10 after
419 const content::NativeWebKeyboardEvent& event, 419 const content::NativeWebKeyboardEvent& event,
420 bool* is_keyboard_shortcut) OVERRIDE; 420 bool* is_keyboard_shortcut) OVERRIDE;
421 virtual void HandleKeyboardEvent( 421 virtual void HandleKeyboardEvent(
422 const content::NativeWebKeyboardEvent& event) OVERRIDE; 422 const content::NativeWebKeyboardEvent& event) OVERRIDE;
423 423
424 // RenderViewHostManager::Delegate ------------------------------------------- 424 // RenderViewHostManager::Delegate -------------------------------------------
425 425
426 virtual bool CreateRenderViewForRenderManager( 426 virtual bool CreateRenderViewForRenderManager(
427 content::RenderViewHost* render_view_host, int opener_route_id) OVERRIDE; 427 content::RenderViewHost* render_view_host, int opener_route_id) OVERRIDE;
428 virtual void BeforeUnloadFiredFromRenderManager( 428 virtual void BeforeUnloadFiredFromRenderManager(
429 bool proceed, 429 bool proceed, const base::TimeTicks& proceed_time,
430 bool* proceed_to_fire_unload) OVERRIDE; 430 bool* proceed_to_fire_unload) OVERRIDE;
431 virtual void RenderViewGoneFromRenderManager( 431 virtual void RenderViewGoneFromRenderManager(
432 content::RenderViewHost* render_view_host) OVERRIDE; 432 content::RenderViewHost* render_view_host) OVERRIDE;
433 virtual void UpdateRenderViewSizeForRenderManager() OVERRIDE; 433 virtual void UpdateRenderViewSizeForRenderManager() OVERRIDE;
434 virtual void NotifySwappedFromRenderManager() OVERRIDE; 434 virtual void NotifySwappedFromRenderManager() OVERRIDE;
435 virtual int CreateOpenerRenderViewsForRenderManager( 435 virtual int CreateOpenerRenderViewsForRenderManager(
436 content::SiteInstance* instance) OVERRIDE; 436 content::SiteInstance* instance) OVERRIDE;
437 virtual NavigationControllerImpl& GetControllerForRenderManager() OVERRIDE; 437 virtual NavigationControllerImpl& GetControllerForRenderManager() OVERRIDE;
438 virtual WebUIImpl* CreateWebUIForRenderManager(const GURL& url) OVERRIDE; 438 virtual WebUIImpl* CreateWebUIForRenderManager(const GURL& url) OVERRIDE;
439 virtual content::NavigationEntry* 439 virtual content::NavigationEntry*
(...skipping 352 matching lines...) Expand 10 before | Expand all | Expand 10 after
792 // If this tab was created from a renderer using window.open, this will be 792 // If this tab was created from a renderer using window.open, this will be
793 // non-NULL and represent the WebUI of the opening renderer. 793 // non-NULL and represent the WebUI of the opening renderer.
794 content::WebUI::TypeID opener_web_ui_type_; 794 content::WebUI::TypeID opener_web_ui_type_;
795 795
796 // The time that we started to create the new tab page. 796 // The time that we started to create the new tab page.
797 base::TimeTicks new_tab_start_time_; 797 base::TimeTicks new_tab_start_time_;
798 798
799 // The time that we started to close this WebContents. 799 // The time that we started to close this WebContents.
800 base::TimeTicks close_start_time_; 800 base::TimeTicks close_start_time_;
801 801
802 // The time when onbeforeunload ended.
803 base::TimeTicks before_unload_end_time_;
804
802 // The time that this tab was last selected. 805 // The time that this tab was last selected.
803 base::TimeTicks last_selected_time_; 806 base::TimeTicks last_selected_time_;
804 807
805 // See description above setter. 808 // See description above setter.
806 bool closed_by_user_gesture_; 809 bool closed_by_user_gesture_;
807 810
808 // Minimum/maximum zoom percent. 811 // Minimum/maximum zoom percent.
809 int minimum_zoom_percent_; 812 int minimum_zoom_percent_;
810 int maximum_zoom_percent_; 813 int maximum_zoom_percent_;
811 // If true, the default zoom limits have been overriden for this tab, in which 814 // If true, the default zoom limits have been overriden for this tab, in which
(...skipping 16 matching lines...) Expand all
828 content::NotificationRegistrar registrar_; 831 content::NotificationRegistrar registrar_;
829 832
830 // Used during IPC message dispatching so that the handlers can get a pointer 833 // Used during IPC message dispatching so that the handlers can get a pointer
831 // to the RVH through which the message was received. 834 // to the RVH through which the message was received.
832 content::RenderViewHost* message_source_; 835 content::RenderViewHost* message_source_;
833 836
834 DISALLOW_COPY_AND_ASSIGN(WebContentsImpl); 837 DISALLOW_COPY_AND_ASSIGN(WebContentsImpl);
835 }; 838 };
836 839
837 #endif // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 840 #endif // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
OLDNEW
« no previous file with comments | « content/browser/web_contents/render_view_host_manager.cc ('k') | content/browser/web_contents/web_contents_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698