Commit 2204ed53 authored by Fabio Tirelo's avatar Fabio Tirelo Committed by Commit Bot

[af] Renable NoAutofillForReadOnlyFields that is no longer flaky

Bug: 834768
Change-Id: Ic51bcfdc7e63baca348c405f7e1f13c06d95b1f3
Reviewed-on: https://chromium-review.googlesource.com/1027899Reviewed-by: default avatarSebastien Seguin-Gagnon <sebsg@chromium.org>
Commit-Queue: Fabio Tirelo <ftirelo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553568}
parent 733c4fa5
...@@ -1536,16 +1536,7 @@ IN_PROC_BROWSER_TEST_P(AutofillInteractiveTest, ComparePhoneNumbers) { ...@@ -1536,16 +1536,7 @@ IN_PROC_BROWSER_TEST_P(AutofillInteractiveTest, ComparePhoneNumbers) {
} }
// Test that Autofill does not fill in read-only fields. // Test that Autofill does not fill in read-only fields.
// Flaky on the official cros-trunk. crbug.com/516052 IN_PROC_BROWSER_TEST_P(AutofillInteractiveTest, NoAutofillForReadOnlyFields) {
// TODO(crbug.com/834768): Check if this is also flaky on other platforms after
// re-enabling it on ChromeOS.
#if defined(OFFICIAL_BUILD)
#define MAYBE_NoAutofillForReadOnlyFields DISABLED_NoAutofillForReadOnlyFields
#else
#define MAYBE_NoAutofillForReadOnlyFields NoAutofillForReadOnlyFields
#endif // defined(OFFICIAL_BUILD)
IN_PROC_BROWSER_TEST_P(AutofillInteractiveTest,
MAYBE_NoAutofillForReadOnlyFields) {
std::string addr_line1("1234 H St."); std::string addr_line1("1234 H St.");
AutofillProfile profile; AutofillProfile profile;
......
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