Commit 7c1d37a9 authored by David Bokan's avatar David Bokan Committed by Commit Bot

Remove DCHECK in scroll metrics recording

This is causing crashes in debug builds. Comment out for now so that it
allows running with DCHECKs enabled while we investigate.

Bug: 944706
Change-Id: I4ff0d604dc52c8893898195c6843e95d91b5598a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1617881
Commit-Queue: Navid Zolghadr <nzolghadr@chromium.org>
Auto-Submit: David Bokan <bokan@chromium.org>
Reviewed-by: default avatarNavid Zolghadr <nzolghadr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#660945}
parent ac586889
......@@ -381,8 +381,9 @@ void ScrollManager::ComputeScrollRelatedMetrics(
if (!scrollable_area || !scrollable_area->ScrollsOverflow())
continue;
DCHECK(!scrollable_area->UsesCompositedScrolling() ||
!scrollable_area->GetNonCompositedMainThreadScrollingReasons());
// TODO(bokan): This DCHECK is occasionally tripped. See crbug.com/944706.
// DCHECK(!scrollable_area->UsesCompositedScrolling() ||
// !scrollable_area->GetNonCompositedMainThreadScrollingReasons());
*non_composited_main_thread_scrolling_reasons |=
scrollable_area->GetNonCompositedMainThreadScrollingReasons();
}
......
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