Commit 08abdaa4 authored by Dirk Pranke's avatar Dirk Pranke Committed by Commit Bot

Revert "Reland "Fix OOPIF hit testing in layout tests.""

This reverts commit 3dcee047.

Reason for revert: Suspect this is causing more flaky failures, see e.g.: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-chromeos-rel/10567 and build 10568

Original change's description:
> Reland "Fix OOPIF hit testing in layout tests."
> 
> This is a reland of 5e93ea19
> 
> Original change's description:
> > Fix OOPIF hit testing in layout tests.
> >
> > If we haven't received an activated surface yet ensure that we fallback
> > to querying the renderer for the target of input events.
> >
> > BUG=601584
> >
> > Change-Id: Id5dfcdb6bf54b98bebbd2a3b715d77202f402781
> > Reviewed-on: https://chromium-review.googlesource.com/1131219
> > Reviewed-by: Ken Buchanan <kenrb@chromium.org>
> > Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#573789}
> 
> TBR=kenrb@chromium.org
> 
> Bug: 601584
> Change-Id: I8a2d207db79cc237e1d5e22e5e9bfffb218486c4
> Reviewed-on: https://chromium-review.googlesource.com/1133718
> Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
> Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#574304}

TBR=kenrb@chromium.org,dtapuska@chromium.org

Change-Id: I9fec8992b35943f8dd2d8ce6bafe8ac9feb49411
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 601584
Reviewed-on: https://chromium-review.googlesource.com/1134143Reviewed-by: default avatarDirk Pranke <dpranke@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#574392}
parent 896be1d3
...@@ -537,8 +537,6 @@ viz::FrameSinkId RenderWidgetHostViewBase::FrameSinkIdAtPoint( ...@@ -537,8 +537,6 @@ viz::FrameSinkId RenderWidgetHostViewBase::FrameSinkIdAtPoint(
gfx::ConvertPointToPixel(device_scale_factor, point); gfx::ConvertPointToPixel(device_scale_factor, point);
viz::SurfaceId surface_id = GetCurrentSurfaceId(); viz::SurfaceId surface_id = GetCurrentSurfaceId();
if (!surface_id.is_valid()) { if (!surface_id.is_valid()) {
// Force a query of the renderer if we don't have a surface id yet.
*out_query_renderer = true;
return GetFrameSinkId(); return GetFrameSinkId();
} }
viz::SurfaceHittest hittest(delegate, viz::SurfaceHittest hittest(delegate,
......
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