Commit 56ebd19d authored by Sky Malice's avatar Sky Malice Committed by Commit Bot

[Feed] Replace feature hack workaround with clang-format off

Bug: 866123
Change-Id: I4483a80d0c81f1634c5867f62a22c43af355d162
Reviewed-on: https://chromium-review.googlesource.com/c/1277812Reviewed-by: default avatarFilip Gorski <fgorski@chromium.org>
Commit-Queue: Sky Malice <skym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599653}
parent e71535af
...@@ -13,13 +13,13 @@ import org.junit.Rule; ...@@ -13,13 +13,13 @@ import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.chromium.base.CommandLine;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.CommandLineFlags; import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.chrome.browser.ChromeFeatureList; import org.chromium.chrome.browser.ChromeFeatureList;
import org.chromium.chrome.browser.ChromeSwitches; import org.chromium.chrome.browser.ChromeSwitches;
import org.chromium.chrome.test.ChromeJUnit4ClassRunner; import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
import org.chromium.chrome.test.ChromeTabbedActivityTestRule; import org.chromium.chrome.test.ChromeTabbedActivityTestRule;
import org.chromium.chrome.test.util.browser.Features;
import org.chromium.components.background_task_scheduler.BackgroundTaskScheduler; import org.chromium.components.background_task_scheduler.BackgroundTaskScheduler;
import org.chromium.components.background_task_scheduler.BackgroundTaskSchedulerFactory; import org.chromium.components.background_task_scheduler.BackgroundTaskSchedulerFactory;
import org.chromium.components.background_task_scheduler.TaskIds; import org.chromium.components.background_task_scheduler.TaskIds;
...@@ -31,10 +31,14 @@ import java.util.List; ...@@ -31,10 +31,14 @@ import java.util.List;
/** /**
* Unit tests for {@link FeedSchedulerBridge}. * Unit tests for {@link FeedSchedulerBridge}.
*/ */
// TODO(https://crbug.com/894334): Remove format suppression once formatting bug is fixed.
// clang-format off
@RunWith(ChromeJUnit4ClassRunner.class) @RunWith(ChromeJUnit4ClassRunner.class)
@CommandLineFlags.Add(ChromeSwitches.DISABLE_FIRST_RUN_EXPERIENCE) @CommandLineFlags.Add(ChromeSwitches.DISABLE_FIRST_RUN_EXPERIENCE)
//@Features.EnableFeatures(ChromeFeatureList.INTEREST_FEED_CONTENT_SUGGESTIONS) @Features.EnableFeatures(ChromeFeatureList.INTEREST_FEED_CONTENT_SUGGESTIONS)
public class FeedRefreshTaskTest { public class FeedRefreshTaskTest {
// clang-format on
@Rule @Rule
public ChromeTabbedActivityTestRule mActivityTestRule = new ChromeTabbedActivityTestRule(); public ChromeTabbedActivityTestRule mActivityTestRule = new ChromeTabbedActivityTestRule();
...@@ -72,13 +76,6 @@ public class FeedRefreshTaskTest { ...@@ -72,13 +76,6 @@ public class FeedRefreshTaskTest {
@Before @Before
public void setUp() throws InterruptedException { public void setUp() throws InterruptedException {
// TODO(https://crbug.com/894334): Replace with @Features.EnableFeatures when presubmit
// forced format is fixed.
CommandLine commandLine = CommandLine.getInstance();
String oldValue = commandLine.getSwitchValue("enable-features");
commandLine.appendSwitchWithValue("enable-features",
oldValue + "," + ChromeFeatureList.INTEREST_FEED_CONTENT_SUGGESTIONS);
mTaskScheduler = new TestBackgroundTaskScheduler(); mTaskScheduler = new TestBackgroundTaskScheduler();
BackgroundTaskSchedulerFactory.setSchedulerForTesting(mTaskScheduler); BackgroundTaskSchedulerFactory.setSchedulerForTesting(mTaskScheduler);
......
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