Commit 8d41aa29 authored by Fady Samuel's avatar Fady Samuel Committed by Commit Bot

Surface synchronization: Report whether an OOPIF is violating invariants

This CL reports whether an OOPIF is causing invariants violations.

Bug: 791154, 672962
Change-Id: Ida943b8a6bb21cd5baed00bd0065f77e5a358033
Reviewed-on: https://chromium-review.googlesource.com/923041
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Reviewed-by: default avatarFady Samuel <fsamuel@chromium.org>
Reviewed-by: default avatarSaman Sami <samans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537600}
parent f7585e8f
......@@ -2733,7 +2733,11 @@ void RenderWidgetHostImpl::SubmitCompositorFrame(
"surface-invariants-violation", base::debug::CrashKeySize::Size256);
base::debug::ScopedCrashKeyString key_value(
crash_key,
new_surface_properties.ToDiffString(last_surface_properties_));
base::StringPrintf(
"[OOPIF? %d] %s\n",
view_ && view_->IsRenderWidgetHostViewChildFrame(),
new_surface_properties.ToDiffString(last_surface_properties_)
.c_str()));
bad_message::ReceivedBadMessage(
GetProcess(), bad_message::RWH_SURFACE_INVARIANTS_VIOLATION);
return;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment