Findit has detected flake occurrences for the test VirtualKeyboardAppWindowTest.DisableOverscrollForImeWindow Culprit (94.8% confidence): https://chromium-review.googlesource.com/q/Id2889577616823cef99e3a3cfea16b5e33a9147c Analysis: https://findit-for-me.appspot.com/waterfall/flake?key=ag9zfmZpbmRpdC1mb3ItbWVywQELEhdNYXN0ZXJGbGFrZUFuYWx5c2lzUm9vdCKKAWNocm9taXVtLmNocm9taXVtb3MvbGludXgtY2hyb21lb3MtZGJnLzc2NDAvYnJvd3Nlcl90ZXN0cy9WbWx5ZEhWaGJFdGxlV0p2WVhKa1FYQndWMmx1Wkc5M1ZHVnpkQzVFYVhOaFlteGxUM1psY25OamNtOXNiRVp2Y2tsdFpWZHBibVJ2ZHc9PQwLEhNNYXN0ZXJGbGFrZUFuYWx5c2lzGAEM Please revert the culprit, or disable the test and find the appropriate owner. If the culprit above is wrong, please file a bug using this link and hit submit: https://bugs.chromium.org/p/chromium/issues/entry?status=Unconfirmed&labels=Pri-1,Test-Findit-Wrong&components=Tools%3ETest%3EFindit%3EFlakiness&summary=%5BFindit%5D%20Flake%20Analyzer%20-%20Wrong%20result%20for%20VirtualKeyboardAppWindowTest.DisableOverscrollForImeWindow&comment=Link%20to%20Analysis%3A%20https://findit-for-me.appspot.com/waterfall/flake?key=ag9zfmZpbmRpdC1mb3ItbWVywQELEhdNYXN0ZXJGbGFrZUFuYWx5c2lzUm9vdCKKAWNocm9taXVtLmNocm9taXVtb3MvbGludXgtY2hyb21lb3MtZGJnLzc2NDAvYnJvd3Nlcl90ZXN0cy9WbWx5ZEhWaGJFdGxlV0p2WVhKa1FYQndWMmx1Wkc5M1ZHVnpkQzVFYVhOaFlteGxUM1psY25OamNtOXNiRVp2Y2tsdFpWZHBibVJ2ZHc9PQwLEhNNYXN0ZXJGbGFrZUFuYWx5c2lzGAEM
From https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-chromeos-dbg/7640 Other Keyboard-related tests might be affected as well (linux-chromeos-dbg). Going to revert the CL.
CL for revert: https://chromium-review.googlesource.com/c/chromium/src/+/1203880
Thanks for the quick fix, working on it now.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a commit b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a Author: Leo Zhang <googleo@chromium.org> Date: Mon Sep 10 06:27:27 2018 Add a ChromeOS feature for IME service. The feature was previously created under the IME service, I refactored it by moving it to ChromeOS features for a simple structure and passing it to the IME extension. This feature is used for switching whether the IME component extension is able to connect to IME service. Bug: 837156, 880245 Change-Id: I832b60e1bcc90708771a935dc9f3172067eb8544 Reviewed-on: https://chromium-review.googlesource.com/1209131 Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org> Reviewed-by: Darren Shen <shend@chromium.org> Commit-Queue: Leo Zhang <googleo@chromium.org> Cr-Commit-Position: refs/heads/master@{#589835} [modify] https://crrev.com/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a/chrome/browser/extensions/BUILD.gn [modify] https://crrev.com/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a/chrome/browser/extensions/api/virtual_keyboard_private/chrome_virtual_keyboard_delegate.cc [modify] https://crrev.com/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a/chrome/browser/extensions/chrome_extensions_interface_registration.cc [modify] https://crrev.com/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a/chromeos/chromeos_features.cc [modify] https://crrev.com/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a/chromeos/chromeos_features.h [modify] https://crrev.com/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a/chromeos/services/ime/public/cpp/BUILD.gn [delete] https://crrev.com/4ecb0a89f46ad7412ae19cc5d7417c23d8b61fab/chromeos/services/ime/public/cpp/features.cc [delete] https://crrev.com/4ecb0a89f46ad7412ae19cc5d7417c23d8b61fab/chromeos/services/ime/public/cpp/features.h [modify] https://crrev.com/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a/ui/keyboard/BUILD.gn [modify] https://crrev.com/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a/ui/keyboard/DEPS [modify] https://crrev.com/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a/ui/keyboard/keyboard_util.cc [modify] https://crrev.com/b5ad005bbf38388596ac7f93bcb3c4d6aae41d8a/ui/keyboard/keyboard_util.h
Comment 1 by huangs@google.com
, Sep 4Owner: goog...@chromium.org