Commit dd2bfe66 authored by Tao Bai's avatar Tao Bai Committed by Commit Bot

Autofill: Fix the failures on O platform

The event sequence was changed by the previous patch
https://chromium-review.googlesource.com/c/chromium/src/+/2296487

The AUTOFILL_VIEW_EXITED is only expected on Android o platform,
and will be removed for others.

This issue wasn't uncoverred because we don't have O trybot by
default.

Bug: 1106457
Change-Id: Id838f7085ae49fd8d968929a7a08d7918c818770
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302743
Commit-Queue: Dominic Battré <battre@chromium.org>
Reviewed-by: default avatarDominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#789568}
parent ea9195f6
...@@ -647,7 +647,7 @@ public class AwAutofillTest { ...@@ -647,7 +647,7 @@ public class AwAutofillTest {
mTest.executeJavaScriptAndWaitForResult("document.getElementById('text2').select();"); mTest.executeJavaScriptAndWaitForResult("document.getElementById('text2').select();");
mTest.dispatchDownAndUpKeyEvents(KeyEvent.KEYCODE_A); mTest.dispatchDownAndUpKeyEvents(KeyEvent.KEYCODE_A);
mCnt += mTest.waitForCallbackAndVerifyTypes(mCnt, mCnt += mTest.waitForCallbackAndVerifyTypes(mCnt,
new Integer[] {AUTOFILL_CANCEL, AUTOFILL_VIEW_EXITED, AUTOFILL_VIEW_ENTERED, new Integer[] {AUTOFILL_VIEW_EXITED, AUTOFILL_CANCEL, AUTOFILL_VIEW_ENTERED,
AUTOFILL_SESSION_STARTED, AUTOFILL_VALUE_CHANGED}); AUTOFILL_SESSION_STARTED, AUTOFILL_VALUE_CHANGED});
} }
...@@ -1126,7 +1126,7 @@ public class AwAutofillTest { ...@@ -1126,7 +1126,7 @@ public class AwAutofillTest {
executeJavaScriptAndWaitForResult("document.getElementById('text2').select();"); executeJavaScriptAndWaitForResult("document.getElementById('text2').select();");
dispatchDownAndUpKeyEvents(KeyEvent.KEYCODE_A); dispatchDownAndUpKeyEvents(KeyEvent.KEYCODE_A);
waitForCallbackAndVerifyTypes(cnt, waitForCallbackAndVerifyTypes(cnt,
new Integer[] {AUTOFILL_CANCEL, AUTOFILL_VIEW_EXITED, AUTOFILL_VIEW_ENTERED, new Integer[] {AUTOFILL_VIEW_EXITED, AUTOFILL_CANCEL, AUTOFILL_VIEW_ENTERED,
AUTOFILL_SESSION_STARTED, AUTOFILL_VALUE_CHANGED}); AUTOFILL_SESSION_STARTED, AUTOFILL_VALUE_CHANGED});
} }
......
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