Issue 682199 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	shuchen@chromium.org
Closed:	Feb 2017
Cc:	ligim...@chromium.org
Components:	UI>Input>Text>IME
EstimatedDays:	----
NextAction:	----
OS:	Linux , Windows
Pri:	1
Type:	Bug
M-56

[Missing Tests]: Makes delay commitText/setComposition can deal with empty text/composition.

Project Member Reported by tkonch...@chromium.org, Jan 18 2017

Issue description

shuchen@, Automated tests for the below commit have been missing and would be really great to add them ASAP since these issues are high priority regressions.

Commit:  https://codereview.chromium.org/2525723005

Ref. Bug: https://bugs.chromium.org/p/chromium/issues/detail?id=668036

Thanks,

Project Member

Comment 1 by bugdroid1@chromium.org, Feb 21 2017

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

commit c49d32e2554c23beb02be20061c0b9784476fe98
Author: shuchen <shuchen@chromium.org>
Date: Tue Feb 21 06:18:39 2017

Adds a test for setting empty composition text.

BUG= 682199 

Review-Url: https://codereview.chromium.org/2710603002
Cr-Commit-Position: refs/heads/master@{#451708}

[modify] https://crrev.com/c49d32e2554c23beb02be20061c0b9784476fe98/chrome/browser/ui/views/ime/input_ime_apitest_nonchromeos.cc
[modify] https://crrev.com/c49d32e2554c23beb02be20061c0b9784476fe98/chrome/test/data/extensions/api_test/input_ime_nonchromeos/background.js
[modify] https://crrev.com/c49d32e2554c23beb02be20061c0b9784476fe98/ui/base/ime/dummy_text_input_client.cc
[modify] https://crrev.com/c49d32e2554c23beb02be20061c0b9784476fe98/ui/base/ime/dummy_text_input_client.h

Comment 2 by shuchen@chromium.org, Feb 21 2017

Status: Fixed (was: Assigned)

►

Issue 682199 link

Issue metadata

[Missing Tests]: Makes delay commitText/setComposition can deal with empty text/composition.

Issue description

Comment 1 by bugdroid1@chromium.org, Feb 21 2017

Comment 1 Deleted

Comment 2 by shuchen@chromium.org, Feb 21 2017

Comment 2 Deleted

Sign in to add a comment