single_process_mash_browser_tests failing on multiple builders |
|||
Issue descriptionFiled by sheriff-o-matic@appspot.gserviceaccount.com on behalf of hajimehoshi@chromium.org single_process_mash_browser_tests failing on multiple builders Builders failed on: - linux-chromeos-dbg: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-chromeos-dbg - Linux Chromium OS ASan LSan Tests (1): https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29
,
Jan 9
https://chromium-review.googlesource.com/c/chromium/src/+/1394595 seems the culprit.
,
Jan 9
,
Jan 9
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/59ec9ad1528347e6b2d7146966437bb89c5e9169 commit 59ec9ad1528347e6b2d7146966437bb89c5e9169 Author: Jun Mukai <mukai@chromium.org> Date: Wed Jan 09 22:42:01 2019 Reland "wait properly for the keyboard bounds change" Instead of RunUntilIdle, it should wait for OnKeyboardOccludedBoundsChange. This CL also allows to resize the window which remotely hosts the VK web-contents based on window.resizeTo(). This is a reland of crrev.com/c/1394595 with an addition of waiting the processing of SetBounds in MockEnableIMEInDifferentExtension(). TBR=stevenjb@chromium.org, xiyuan@chromium.org Bug: 916177, 920073 Test: browser_tests Change-Id: Ib353f5193ab4f7bb907a6c02ece6429b42408919 Reviewed-on: https://chromium-review.googlesource.com/c/1403923 Reviewed-by: Jun Mukai <mukai@chromium.org> Commit-Queue: Jun Mukai <mukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#621343} [modify] https://crrev.com/59ec9ad1528347e6b2d7146966437bb89c5e9169/ash/keyboard/ash_keyboard_ui.cc [modify] https://crrev.com/59ec9ad1528347e6b2d7146966437bb89c5e9169/chrome/browser/ui/ash/keyboard/keyboard_controller_browsertest.cc
,
Jan 10
seems reland is okay. |
|||
►
Sign in to add a comment |
|||
Comment 1 by hajimehoshi@chromium.org
, Jan 9Labels: OS-Chrome