New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 732402 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Aug 15
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug


Sign in to add a comment

Asynchronous immediate mode hit-test inquiry

Project Member Reported by rjkroege@chromium.org, Jun 12 2017

Issue description

There may always exist areas that we wish to target events to that cannot be represented by the current HitTest component implementation. In this case, the event-targeting logic needs to ask the client (e.g. blink, UI) what should be the disposition of the event. Implement this fall-back handling.


 
Cc: nzolghadr@chromium.org
Labels: event-targeting

Comment 2 by bokan@chromium.org, Jun 29 2017

Cc: bokan@chromium.org
Blocking: 732405
Blocking: 732400
Blocking: 732398
Project Member

Comment 6 by bugdroid1@chromium.org, Jan 9 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/08e1c97b0f84b6aa6caef59776d203ef39c901fd

commit 08e1c97b0f84b6aa6caef59776d203ef39c901fd
Author: Ria Jiang <riajiang@chromium.org>
Date: Tue Jan 09 23:41:13 2018

Tell clients to do hit-testing with kHitTestAsk flag in HitTestQuery.

When the target hit-test region has kHitTestAsk flag set, we need to
fall back to do hit-testing on the client side. HitTestQuery returns
an empty target with kHitTestAsk flag and clients will then ask to do
async targeting.

Bug:  732402 
Test: viz_unittests
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I16e17f4d4b7d84ecdefc09d289248115070190ec
Reviewed-on: https://chromium-review.googlesource.com/811836
Reviewed-by: Robert Kroeger <rjkroege@chromium.org>
Commit-Queue: Ria Jiang <riajiang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#528168}
[modify] https://crrev.com/08e1c97b0f84b6aa6caef59776d203ef39c901fd/components/viz/host/hit_test/hit_test_query.cc
[modify] https://crrev.com/08e1c97b0f84b6aa6caef59776d203ef39c901fd/components/viz/host/hit_test/hit_test_query_unittest.cc

Components: -Internals>MUS Internals>Services>WindowService
Labels: -Proj-Mustash-Mus-WS
Deprecating label Proj-Mustash-Mus-WS in favor of Components.
Labels: -Proj-Mustash Proj-Mash-MultiProcess
Bug scrub: Is this needed to ship mustash/mash?

Status: WontFix (was: Assigned)
This is a duplicate of a fixed bug. 

However: there is probably something that needs to be done for event targeting in separate process ash/browser.

Sign in to add a comment