Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(27)

Issue 2594333002: Remove obsolete and flaky ImeTest#testPasteLongText (Closed)

Created:
4 years ago by yabinh
Modified:
4 years ago
CC:
chromium-reviews, jam, darin-cc_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove obsolete and flaky ImeTest#testPasteLongText The purpose of ImeTest#testPasteLongText is to show that Chrome can crash if IME thread is not enabled. Since IME thread is landed, we don't need the test any more. Besides, the test is flaky on low end device because it takes too much time to paste long text. BUG=660312 Committed: https://crrev.com/c4f371e1758fd6c34827e571064b93f319ae312e Cr-Commit-Position: refs/heads/master@{#440357}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -27 lines) Patch
M content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java View 1 chunk +0 lines, -27 lines 0 comments Download

Messages

Total messages: 13 (8 generated)
yabinh
PTAL, thanks!
4 years ago (2016-12-22 07:29:28 UTC) #4
Changwan Ryu
lgtm, thanks!
4 years ago (2016-12-22 07:32:39 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2594333002/1
4 years ago (2016-12-22 07:34:14 UTC) #8
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years ago (2016-12-22 07:49:36 UTC) #11
commit-bot: I haz the power
4 years ago (2016-12-22 07:53:16 UTC) #13
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/c4f371e1758fd6c34827e571064b93f319ae312e
Cr-Commit-Position: refs/heads/master@{#440357}

Powered by Google App Engine
This is Rietveld 408576698