Commit a4bcee53 authored by W. James MacLean's avatar W. James MacLean Committed by Commit Bot

Add logging to understand test timeout.

Adds logging to
SitePerProcessEmulatedTouchBrowserTest.EmulatedTouchShowPressHasTouchID
to verify the gesture stream when the test times out.

Bug: 833380
Change-Id: I8cf2e92a6f13aa38e3294f4511d53ce658cd62a5
Reviewed-on: https://chromium-review.googlesource.com/1187280Reviewed-by: default avatarKen Buchanan <kenrb@chromium.org>
Commit-Queue: James MacLean <wjmaclean@chromium.org>
Cr-Commit-Position: refs/heads/master@{#585587}
parent d2787ed6
...@@ -1247,6 +1247,17 @@ class SitePerProcessEmulatedTouchBrowserTest ...@@ -1247,6 +1247,17 @@ class SitePerProcessEmulatedTouchBrowserTest
[](blink::WebInputEvent::Type expected_type, [](blink::WebInputEvent::Type expected_type,
const gfx::Point& expected_position, content::InputEventAckSource, const gfx::Point& expected_position, content::InputEventAckSource,
content::InputEventAckState, const blink::WebInputEvent& event) { content::InputEventAckState, const blink::WebInputEvent& event) {
#if defined(OS_WIN)
// Add some logging to diagnose a potential source of flake:
// the hypothesis is that something is causing the gesture
// stream to cancel before kGestureShowPress is generated, so
// we'll dump the event stream that we actually see in this case.
// https://crbug.com/833380.
if (expected_type == blink::WebInputEvent::kGestureShowPress) {
LOG(ERROR) << "Waiting for: kGestureShowPress: ack seen for "
<< blink::WebInputEvent::GetName(event.GetType());
}
#endif
if (event.GetType() != expected_type) if (event.GetType() != expected_type)
return false; return false;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment