Chromium Code Reviews| Index: shell/shell_test_base.cc |
| diff --git a/shell/shell_test_base.cc b/shell/shell_test_base.cc |
| index daeef191b883157926c56326cf3203473ae36102..4c616e2430969b29f3452b507456a2a1c18d8604 100644 |
| --- a/shell/shell_test_base.cc |
| +++ b/shell/shell_test_base.cc |
| @@ -24,14 +24,19 @@ ShellTestBase::~ShellTestBase() { |
| } |
| void ShellTestBase::SetUp() { |
| + SetUpTestApplications(); |
| shell_context_.Init(); |
| - base::FilePath service_dir; |
| - CHECK(PathService::Get(base::DIR_MODULE, &service_dir)); |
| } |
| ScopedMessagePipeHandle ShellTestBase::ConnectToService( |
| const GURL& application_url, |
| const std::string& service_name) { |
| + return shell_context_.ConnectToServiceByName(application_url, service_name) |
| + .Pass(); |
| +} |
| + |
| +#if !defined(OS_ANDROID) |
| +void ShellTestBase::SetUpTestApplications() { |
|
viettrungluu
2014/12/19 18:31:34
Are you missing a shell_test_base_android.cc from
qsr
2014/12/22 10:52:32
Yes, sorry. Added.
|
| // Set the MojoURLResolver origin to be the same as the base file path for |
| // local files. This is primarily for test convenience, so that references |
| // to unknown mojo: urls that do not have specific local file or custom |
| @@ -40,17 +45,8 @@ ScopedMessagePipeHandle ShellTestBase::ConnectToService( |
| CHECK(PathService::Get(base::DIR_MODULE, &service_dir)); |
| shell_context_.mojo_url_resolver()->SetBaseURL( |
| FilePathToFileURL(service_dir)); |
| - |
| - return shell_context_.ConnectToServiceByName(application_url, service_name) |
| - .Pass(); |
| -} |
| - |
| -ScopedMessagePipeHandle ShellTestBase::ConnectToServiceViaNetwork( |
| - const GURL& application_url, |
| - const std::string& service_name) { |
| - return shell_context_.ConnectToServiceByName(application_url, service_name) |
| - .Pass(); |
| } |
| +#endif |
| } // namespace test |
| } // namespace shell |