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

Unified Diff: content/browser/renderer_host/render_widget_host.h

Issue 8510038: Fix hung renderer timer code in RenderWidgetHost (task spam bug) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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: content/browser/renderer_host/render_widget_host.h
diff --git a/content/browser/renderer_host/render_widget_host.h b/content/browser/renderer_host/render_widget_host.h
index 539ddb3c0df08e8abe6549c0b51c5ac5ed6f4a9d..20899707ce61d9558fa7d724908e1f16b9247785 100644
--- a/content/browser/renderer_host/render_widget_host.h
+++ b/content/browser/renderer_host/render_widget_host.h
@@ -712,11 +712,13 @@ class CONTENT_EXPORT RenderWidgetHost : public IPC::Channel::Listener,
// This is true if the renderer is currently unresponsive.
bool is_unresponsive_;
- // The following value indicates a time in the future when we would consider
- // the renderer hung if it does not generate an appropriate response message.
- base::Time time_when_considered_hung_;
+ // This is true if we are currently monitoring for a hung renderer.
+ bool is_monitoring_for_hung_renderer_;
- // This timer runs to check if time_when_considered_hung_ has past.
+ // Last time we heard from the renderer.
+ base::TimeTicks time_of_last_renderer_response_;
+
+ // This timer runs to check if the renderer is unresponsive.
base::OneShotTimer<RenderWidgetHost> hung_renderer_timer_;
// Flag to detect recursive calls to GetBackingStore().

Powered by Google App Engine
This is Rietveld 408576698