Commit ddd1bb9e authored by Marc Treib's avatar Marc Treib Committed by Commit Bot

Don't unconditionally access verifier() in TwoClientSearchEnginesSyncTest

The verifier profile doesn't always exist; notably it doesn't when
running against external servers (aka E2E_TEST, aka KitchenSync).
This fixes a test crash when running in that mode.

Bug: 1135195
Change-Id: I163583c74e2139cd50f9d658657832d57550a3ae
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2450290
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Auto-Submit: Marc Treib <treib@chromium.org>
Reviewed-by: default avatarMikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#814125}
parent bb2102fa
...@@ -29,8 +29,12 @@ class TwoClientSearchEnginesSyncTest : public SyncTest { ...@@ -29,8 +29,12 @@ class TwoClientSearchEnginesSyncTest : public SyncTest {
} }
// Wait for models to load. // Wait for models to load.
search_test_utils::WaitForTemplateURLServiceToLoad( if (use_verifier()) {
TemplateURLServiceFactory::GetForProfile(verifier())); // Note: The verifier profile isn't used when running against external
// servers.
search_test_utils::WaitForTemplateURLServiceToLoad(
TemplateURLServiceFactory::GetForProfile(verifier()));
}
search_test_utils::WaitForTemplateURLServiceToLoad( search_test_utils::WaitForTemplateURLServiceToLoad(
TemplateURLServiceFactory::GetForProfile(GetProfile(0))); TemplateURLServiceFactory::GetForProfile(GetProfile(0)));
search_test_utils::WaitForTemplateURLServiceToLoad( search_test_utils::WaitForTemplateURLServiceToLoad(
......
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