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

Unified Diff: content/browser/web_contents/web_contents_impl.cc

Issue 2737553003: Remove ChildProcess.HangRendererType. (Closed)
Patch Set: rebase on new version Created 3 years, 9 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: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 22a92be332c23f4fdaa77dca0b306f695a221bbb..6e6bd04d1809e2862f19fa9d9f0d2f7910e7cb44 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -4836,8 +4836,7 @@ void WebContentsImpl::OnIgnoredUIEvent() {
}
void WebContentsImpl::RendererUnresponsive(
- RenderWidgetHostImpl* render_widget_host,
- RendererUnresponsiveType type) {
+ RenderWidgetHostImpl* render_widget_host) {
for (auto& observer : observers_)
observer.OnRendererUnresponsive(render_widget_host);
@@ -4848,17 +4847,11 @@ void WebContentsImpl::RendererUnresponsive(
if (ShouldIgnoreUnresponsiveRenderer())
return;
- // Record histograms about the type of renderer hang.
- UMA_HISTOGRAM_ENUMERATION(
- "ChildProcess.HangRendererType", type,
- RendererUnresponsiveType::RENDERER_UNRESPONSIVE_MAX);
-
if (!GetRenderViewHost() || !GetRenderViewHost()->IsRenderViewLive())
return;
if (delegate_) {
WebContentsUnresponsiveState unresponsive_state;
- unresponsive_state.reason = type;
unresponsive_state.outstanding_ack_count =
render_widget_host->in_flight_event_count();
unresponsive_state.outstanding_event_type =

Powered by Google App Engine
This is Rietveld 408576698