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

Issue 743059 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

VR: change RenderWidgetHost interface to allow routing the event when is needed.

Project Member Reported by asimjour@chromium.org, Jul 14 2017

Issue description

RenderWidgetHost should be able to send events to InputEventRouter.
This is needed to fix the problem in interaction with cross-origin iframes while in VR browsing mode. Moreover, the refactoring will remove repeated code in render_widget_host_view_android, render_widget_host_view_aura, and render_widget_host_view_mac.  Event routing has some similarities between different platforms, it can be merged into 
render_widget_host_impl. At very least the logic for deciding whether should_route_event (https://cs.chromium.org/chromium/src/content/browser/renderer_host/render_widget_host_view_android.cc?dr=CSs&l=1957, https://cs.chromium.org/chromium/src/content/browser/renderer_host/render_widget_host_view_event_handler.cc?dr=CSs&l=373) or not can be merged.

 
Blocking: 740984

Comment 2 by boliu@chromium.org, Jul 14 2017

Components: Internals>Input
I'm not very familiar with input code. anyone from input team has opinions here? 
Cc: sadrul@chromium.org rbyers@chromium.org
+sadrul@ +rbyers@
Labels: -M-61 M-62
asimjour: Is this a P1 for M62?
@ddrorwin. Yes, this is a P1 for M62
Is anyone working on this? What is the impact? Is this really something we need in M62?
Labels: -M-62 M-63
Labels: -Pri-1 Pri-2
Labels: -Pri-2 -M-63 M-64 Pri-1
We need to address this because we need to fix the bug it is blocking.
Labels: -M-64 M-65
Status: WontFix (was: Started)
We don't need this anymore, the input routing is changing with  issue 793430 
Blocking: -740984
Labels: Test-Complete

Sign in to add a comment