Popup touch handle blinks when first long press page
Reported by
fanjins...@sogou-inc.com,
Jan 2 2018
|
||||||||
Issue descriptionSteps to reproduce the problem: 1. Build WebView instrumentation APK by the latest source code, 2. Open https://m.sohu.com, long press the page, 3. Popup touch handle will blink on the left-top page, as the attachment video shows. My device is HUAWEI MATE 10. What is the expected behavior? What went wrong? Popup touch handle will blink on the left-top page, as the attachment video shows. Did this work before? N/A Chrome version: <Copy from: 'about:version'> Channel: stable OS Version: 7.0 Flash Version: Shockwave Flash 28.0 r0
,
Jan 3 2018
@fanjinsong: Thanks for the report!! Could you please attach a sample .apk file where you're seeing this issue and also attach a screencast as well as the above video is not accessible. Thanks!!
,
Jan 3 2018
Hi @sandeepkumars, It's so fast, I can not take a screenshot, but I upload the attachment video on Youtube,https://youtu.be/YGiAJuSms5Y I test Android WebView Test Shell(M62,M63 and the latest) can reproduce this problem.
,
Jan 3 2018
Thank you for providing more feedback. Adding requester "sandeepkumars@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 3 2018
fanjinsong@, thanks for reporting this bug, it looks like a regular selection handle issue not Smart Selection related though. It might be issue 773515 . I'll take a look tomorrow.
,
Jan 3 2018
,
Jan 3 2018
ctzsm@, thanks for confirm, this bug only occurs when the selection menu appears for the first time, might be different from issue 773515 . I do a fix in the local branch, maybe upload CL for review.
,
Jan 3 2018
fanjinsong@, this is not issue 773515 for sure, thanks for fixing this issue.
,
Jan 5 2018
Adding a milestone for tracking purpose.
,
Jan 9 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5b82f2d2aae0ca24b877009200ced9065a772e73 commit 5b82f2d2aae0ca24b877009200ced9065a772e73 Author: Jinsong Fan <fanjinsong@sogou-inc.com> Date: Tue Jan 09 23:17:20 2018 AW: Fix Popup touch handle blinks when first show PopupTouchHandleDrawable's visibility (include mAlpha, mVisible) is inconsistent with the TouchHandle in native. This CL is set to INVISIBLE when PopupTouchHandleDrawable constructed. Bug: 798252 Change-Id: I749834322146c50acc5fa1dd152fd61ff3092dd0 Reviewed-on: https://chromium-review.googlesource.com/848435 Reviewed-by: Shimi Zhang <ctzsm@chromium.org> Reviewed-by: Pedro Amaral <amaralp@chromium.org> Reviewed-by: Tobias Sargeant <tobiasjs@chromium.org> Commit-Queue: Shimi Zhang <ctzsm@chromium.org> Cr-Commit-Position: refs/heads/master@{#528161} [modify] https://crrev.com/5b82f2d2aae0ca24b877009200ced9065a772e73/android_webview/java/src/org/chromium/android_webview/PopupTouchHandleDrawable.java
,
Jan 9 2018
,
Jan 16 2018
Fix issue verified on latest M65:65.0.3322.0 as per#3. Tested device:Nexus 5X / N2G48I
,
Jan 24 2018
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by pnangunoori@chromium.org
, Jan 3 2018