Commit f8cf034f authored by David Bertoni's avatar David Bertoni Committed by Commit Bot

[Extensions] Fix calls to parent SetUp functions in test classes.

This CL fixes one more instance of this issue by moving the code to
the constructor.

Bug: 1117287
Change-Id: I21c1749d1bc037c7952ef7130934c50d37f6b5cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2376044Reviewed-by: default avatarDevlin <rdevlin.cronin@chromium.org>
Commit-Queue: David Bertoni <dbertoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#801788}
parent 1d02a4c1
...@@ -31,10 +31,7 @@ class ExecuteScriptApiTestBase : public ExtensionApiTest { ...@@ -31,10 +31,7 @@ class ExecuteScriptApiTestBase : public ExtensionApiTest {
class ExecuteScriptApiTest : public ExecuteScriptApiTestBase, class ExecuteScriptApiTest : public ExecuteScriptApiTestBase,
public testing::WithParamInterface<ContextType> { public testing::WithParamInterface<ContextType> {
protected: protected:
void SetUp() override { ExecuteScriptApiTest() {
ExecuteScriptApiTestBase::SetUp();
// Service Workers are currently only available on certain channels, so set
// the channel for those tests.
if (GetParam() == ContextType::kServiceWorker) if (GetParam() == ContextType::kServiceWorker)
current_channel_ = std::make_unique<ScopedWorkerBasedExtensionsChannel>(); current_channel_ = std::make_unique<ScopedWorkerBasedExtensionsChannel>();
} }
......
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