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

Issue 818104 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
(currently inactive on Chromium)
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug
Proj-VR
Proj-XR
Proj-XR-VR

Blocking:
issue 641470



Sign in to add a comment

VR: Web input selection broken

Project Member Reported by ymalik@chromium.org, Mar 2 2018

Issue description

1) Spawn the keyboard by clicking on a web input field
2) Enter some text on the input field
3) Double click on text to create a selection
4) Press backspace on the keyboard
Expected behavior: the selected word should be deleted
Actual behavior: the keyboard freezes
 
Components: UI>Browser>VR

Comment 2 by ericde@google.com, Mar 3 2018

Labels: hotlist-vrb-mvp
Status: Started (was: Assigned)
Patch under review: https://chromium-review.googlesource.com/c/chromium/src/+/950684
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 6 2018

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

commit db331fe0cd8a5a25e09a7b1a33d8ea3e0391de66
Author: Yash Malik <ymalik@google.com>
Date: Tue Mar 06 12:38:23 2018

VR: Fix selection and composition diffs

This CL fixes two separate bugs::
1) We weren't sending empty commit messages when the user pressed backspace on
   selected text. As a result, the keyboard would not see an ack and freeze.
2) We incorrectly assumed that we wouldn't see a new composition while there is
   an existing composition. This is not true as the keyboard can coalesce
   events.

Bug:  818104 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ifdb28f06fe445a901b3ef32504f5f2c65b933243
Reviewed-on: https://chromium-review.googlesource.com/950684
Commit-Queue: Ian Vollick <vollick@chromium.org>
Reviewed-by: Ian Vollick <vollick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541082}
[modify] https://crrev.com/db331fe0cd8a5a25e09a7b1a33d8ea3e0391de66/chrome/browser/vr/model/text_input_info.cc
[modify] https://crrev.com/db331fe0cd8a5a25e09a7b1a33d8ea3e0391de66/chrome/browser/vr/model/text_input_info_unittest.cc

Labels: Merge-Request-66
Project Member

Comment 6 by sheriffbot@chromium.org, Mar 8 2018

Labels: -Merge-Request-66 Merge-Approved-66 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M66. Please go ahead and merge the CL to branch 3359 manually. Please contact milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), josafat@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 7 by bugdroid1@chromium.org, Mar 8 2018

Labels: -merge-approved-66 merge-merged-3359
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d2d7bf08b3d1075902348531671b7b4e5f4a84c8

commit d2d7bf08b3d1075902348531671b7b4e5f4a84c8
Author: Yash Malik <ymalik@google.com>
Date: Thu Mar 08 11:23:01 2018

VR: Fix selection and composition diffs

This CL fixes two separate bugs::
1) We weren't sending empty commit messages when the user pressed backspace on
   selected text. As a result, the keyboard would not see an ack and freeze.
2) We incorrectly assumed that we wouldn't see a new composition while there is
   an existing composition. This is not true as the keyboard can coalesce
   events.

Bug:  818104 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ifdb28f06fe445a901b3ef32504f5f2c65b933243
Reviewed-on: https://chromium-review.googlesource.com/950684
Commit-Queue: Ian Vollick <vollick@chromium.org>
Reviewed-by: Ian Vollick <vollick@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#541082}(cherry picked from commit db331fe0cd8a5a25e09a7b1a33d8ea3e0391de66)
Reviewed-on: https://chromium-review.googlesource.com/955362
Reviewed-by: Yash Malik <ymalik@chromium.org>
Cr-Commit-Position: refs/branch-heads/3359@{#89}
Cr-Branched-From: 66afc5e5d10127546cc4b98b9117aff588b5e66b-refs/heads/master@{#540276}
[modify] https://crrev.com/d2d7bf08b3d1075902348531671b7b4e5f4a84c8/chrome/browser/vr/model/text_input_info.cc
[modify] https://crrev.com/d2d7bf08b3d1075902348531671b7b4e5f4a84c8/chrome/browser/vr/model/text_input_info_unittest.cc

Status: Fixed (was: Started)
Labels: Test-Complete
Status: Verified (was: Fixed)
Verified on builds 66.0.3359.28 and 67.0.3368.0  
Test case "Edit text after text selection" to "2D UI Handling Test Plan"

Sign in to add a comment