Chromium Code Reviews| Index: chrome/browser/autofill/autofill_browsertest.cc |
| diff --git a/chrome/browser/autofill/autofill_browsertest.cc b/chrome/browser/autofill/autofill_browsertest.cc |
| index 26c64aacac57d4c448cb8e0637b0768b3fc1622c..80d582d2fc46a5ae6dc90a05385d3b4c35f5699f 100644 |
| --- a/chrome/browser/autofill/autofill_browsertest.cc |
| +++ b/chrome/browser/autofill/autofill_browsertest.cc |
| @@ -155,7 +155,11 @@ class AutofillTest : public InProcessBrowserTest { |
| bool result = false; |
| ASSERT_TRUE(ui_test_utils::ExecuteJavaScriptAndExtractBool( |
| render_view_host(), L"", |
| - L"document.getElementById('firstname').focus();", &result)); |
| + L"if (document.readyState === 'complete')" |
| + L" document.getElementById('firstname').focus();" |
| + L"else" |
| + L" domAutomationController.send(false);", |
| + &result)); |
| ASSERT_TRUE(result); |
| } |
| @@ -384,7 +388,8 @@ IN_PROC_BROWSER_TEST_F(AutofillTest, AutofillFormsDistinguishedById) { |
| // This test brought to you by http://crbug.com/69204 |
| #if defined(OS_MACOSX) |
| // Sometimes times out on Mac: http://crbug.com/81451 |
| -#define MAYBE_AutofillAfterReload DISABLED_AutofillAfterReload |
| +// Currently enabled for logging. |
|
dhollowa
2011/06/08 14:45:17
Better to just delete these macros. You can add t
|
| +#define MAYBE_AutofillAfterReload AutofillAfterReload |
| #else |
| #define MAYBE_AutofillAfterReload AutofillAfterReload |
| #endif |