WebView instrumentation(Android WebView TestShell) can't handle any touch events after drag-and-drop
Reported by
fanjins...@sogou-inc.com,
Jan 4
|
|||||
Issue descriptionSteps to reproduce the problem: 1. Build WebView instrumentation apk(ninja -C out/Default webview_instrumentation_apk) by the source code, and install WebViewInstrumentation.apk on Android N or N+ 2. Load url https://m.sohu.com, and open any news, then select some words by the long press 3. Long press the selected words, then drag the selected word to any place 4. After dropped, TestShell cann't handle any touch events except scroll. What is the expected behavior? What went wrong? TestShell cann't handle any touch events after drag-and-drop. Did this work before? N/A Chrome version: 73.0.3661.0 Channel: stable OS Version: 7.1.1 Flash Version:
,
Jan 4
@fanjinsong: Please provide test file to install or link to download along with screencast on reproducing the issue, this would help in further triaging. Thanks!
,
Jan 7
@jbanavatu This URL: https://dlmse.sogoucdn.com/awp/WebViewInstrumentation.apk links to the WebViewInstrumentation apk. And the attached screencast shows this issue.
,
Jan 7
Thank you for providing more feedback. Adding the requester to the cc list. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 7
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f9476f0bc07d9d735f867a2cb1af4665806fe72d commit f9476f0bc07d9d735f867a2cb1af4665806fe72d Author: Jinsong Fan <fanjinsong@sogou-inc.com> Date: Mon Jan 07 16:48:05 2019 AW: Override onDragEvent() for webview instrumentation Since touch-drag-drop is enabled by default on Android N+, the AwTestContainerView does not pass the DragEvent to the AwContents, causing the blink to not handle the touch event correctly after drag-and-drop. The CL overrides onDragEvent() for webview instrumentation apk. Bug: 919013 Change-Id: I77a31c2243f2ade3b2c17aa14d733bda05c2dc3d Reviewed-on: https://chromium-review.googlesource.com/c/1393130 Reviewed-by: Changwan Ryu <changwan@chromium.org> Commit-Queue: Changwan Ryu <changwan@chromium.org> Cr-Commit-Position: refs/heads/master@{#620354} [modify] https://crrev.com/f9476f0bc07d9d735f867a2cb1af4665806fe72d/android_webview/test/shell/src/org/chromium/android_webview/test/AwTestContainerView.java
,
Jan 8
Unable to assign to fanjinsong@sogou-inc.com, hence assigning to reviewer changwan@. Please change if this is not the case. @fanjinsong@sogou-inc.com: Could you please let us know if this is fixed in latest canary, we can change status to fixed. Thanks!
,
Jan 8
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by jbanavatu@chromium.org
, Jan 4