| Index: chrome/browser/safe_browsing/safe_browsing_test.cc
|
| ===================================================================
|
| --- chrome/browser/safe_browsing/safe_browsing_test.cc (revision 66118)
|
| +++ chrome/browser/safe_browsing/safe_browsing_test.cc (working copy)
|
| @@ -289,6 +289,10 @@
|
| // Makes sure the auto update is not triggered. This test will force the
|
| // update when needed.
|
| command_line->AppendSwitch(switches::kSbDisableAutoUpdate);
|
| + // This test uses loopback. No need to use IPv6 especially it makes
|
| + // local requests slow on Windows trybot when ipv6 local address [::1]
|
| + // is not setup.
|
| + command_line->AppendSwitch(switches::kDisableIPv6);
|
|
|
| // In this test, we fetch SafeBrowsing data and Mac key from the same
|
| // server. Although in real production, they are served from different
|
| @@ -554,15 +558,6 @@
|
| EXPECT_TRUE(is_database_ready());
|
| EXPECT_FALSE(is_update_scheduled());
|
|
|
| - // TODO(lzheng): Remove the following #if and #elif to enable the rest of
|
| - // the test once bot is restarted with change
|
| - // http://codereview.chromium.org/3750002.
|
| -#if defined(OS_WIN)
|
| - break;
|
| -#elif defined(OS_POSIX)
|
| - if (step > 2 ) break;
|
| -#endif
|
| -
|
| // Starts safebrowsing update on IO thread. Waits till scheduled
|
| // update finishes. Stops waiting after kMaxWaitSecPerStep if the update
|
| // could not finish.
|
| @@ -623,14 +618,11 @@
|
| last_step = step;
|
| }
|
|
|
| - // TODO(lzheng): Enable this check after safebrowsing server updated with
|
| - // the latest data in the next revision.
|
| -
|
| // Verifies with server if test is done and waits till server responses.
|
| - // EXPECT_EQ(URLRequestStatus::SUCCESS,
|
| - // safe_browsing_helper->VerifyTestComplete(server_host,
|
| - // server_port,
|
| - // last_step));
|
| - // EXPECT_EQ("yes", safe_browsing_helper->response_data());
|
| + EXPECT_EQ(URLRequestStatus::SUCCESS,
|
| + safe_browsing_helper->VerifyTestComplete(server_host,
|
| + server_port,
|
| + last_step));
|
| + EXPECT_EQ("yes", safe_browsing_helper->response_data());
|
| test_server.Stop();
|
| }
|
|
|