Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(476)

Side by Side Diff: chrome/browser/autofill/autofill_interactive_uitest.cc

Issue 2343643002: [Autofill] Disable AutofillInteractiveTest.AutofillEvents on ChromeOS. (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <string> 5 #include <string>
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/files/file_util.h" 8 #include "base/files/file_util.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/memory/ref_counted.h" 10 #include "base/memory/ref_counted.h"
(...skipping 1261 matching lines...) Expand 10 before | Expand all | Expand 10 after
1272 content::WebContents* web_contents = GetWebContents(); 1272 content::WebContents* web_contents = GetWebContents();
1273 web_contents->GetController().Reload(false); 1273 web_contents->GetController().Reload(false);
1274 content::WaitForLoadStop(web_contents); 1274 content::WaitForLoadStop(web_contents);
1275 1275
1276 // Invoke Autofill. 1276 // Invoke Autofill.
1277 TryBasicFormFill(); 1277 TryBasicFormFill();
1278 } 1278 }
1279 1279
1280 // Test that filling a form sends all the expected events to the different 1280 // Test that filling a form sends all the expected events to the different
1281 // fields being filled. 1281 // fields being filled.
1282 IN_PROC_BROWSER_TEST_F(AutofillInteractiveTest, AutofillEvents) { 1282 // Flakily fails on ChromeOS (crbug.com/646576).
1283 #if defined(OS_CHROMEOS)
1284 #define MAYBE_AutofillEvents DISABLED_AutofillEvents
1285 #else
1286 #define MAYBE_AutofillEvents AutofillEvents
1287 #endif
1288 IN_PROC_BROWSER_TEST_F(AutofillInteractiveTest, MAYBE_AutofillEvents) {
1283 CreateTestProfile(); 1289 CreateTestProfile();
1284 1290
1285 // Load the test page. 1291 // Load the test page.
1286 ASSERT_NO_FATAL_FAILURE(ui_test_utils::NavigateToURL( 1292 ASSERT_NO_FATAL_FAILURE(ui_test_utils::NavigateToURL(
1287 browser(), GURL(std::string(kDataURIPrefix) + kTestEventFormString))); 1293 browser(), GURL(std::string(kDataURIPrefix) + kTestEventFormString)));
1288 1294
1289 // Invoke Autofill. 1295 // Invoke Autofill.
1290 TryBasicFormFill(); 1296 TryBasicFormFill();
1291 1297
1292 // Checks that all the events were fired for the input field. 1298 // Checks that all the events were fired for the input field.
(...skipping 422 matching lines...) Expand 10 before | Expand all | Expand 10 after
1715 ASSERT_NO_FATAL_FAILURE(ui_test_utils::NavigateToURL(browser(), 1721 ASSERT_NO_FATAL_FAILURE(ui_test_utils::NavigateToURL(browser(),
1716 GURL(std::string(kDataURIPrefix) + kTestPasswordFormString))); 1722 GURL(std::string(kDataURIPrefix) + kTestPasswordFormString)));
1717 ASSERT_TRUE(content::ExecuteScript( 1723 ASSERT_TRUE(content::ExecuteScript(
1718 GetRenderViewHost(), 1724 GetRenderViewHost(),
1719 "document.getElementById('user').value = 'user';")); 1725 "document.getElementById('user').value = 'user';"));
1720 FocusFieldByName("password"); 1726 FocusFieldByName("password");
1721 PasteStringAndWait("foobar"); 1727 PasteStringAndWait("foobar");
1722 } 1728 }
1723 1729
1724 } // namespace autofill 1730 } // namespace autofill
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698