Commit e17ebfdc authored by Etienne Bergeron's avatar Etienne Bergeron Committed by Commit Bot

Reactivate thread priority checks in ChromeOS tests

Thread priority should not be modified after a test.
If that is happening, it needs to be fixed in the code.

This CL remove the code that disable the priority checks.

NOTE TO SHERIFF:
 * please paste failure logs when reverting, the goal is
   to remove flaky tests related to thread priority.
 * assigned flaky test to etienneb@

Bug: 1097092
Change-Id: I086ceb10e88a657c91d8d4938e49ee85f7d66f7a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2295082Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
Commit-Queue: Etienne Bergeron <etienneb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#787791}
parent 801cfe88
...@@ -30,10 +30,8 @@ class BrowserTestSuiteRunnerChromeOS : public ChromeTestSuiteRunner { ...@@ -30,10 +30,8 @@ class BrowserTestSuiteRunnerChromeOS : public ChromeTestSuiteRunner {
public: public:
int RunTestSuite(int argc, char** argv) override { int RunTestSuite(int argc, char** argv) override {
BrowserTestSuiteChromeOS test_suite(argc, argv); BrowserTestSuiteChromeOS test_suite(argc, argv);
// Browser tests are expected not to tear-down various globals and may // Browser tests are expected not to tear-down various globals.
// complete with the thread priority being above NORMAL.
test_suite.DisableCheckForLeakedGlobals(); test_suite.DisableCheckForLeakedGlobals();
test_suite.DisableCheckForThreadPriorityAtTestEnd();
return test_suite.Run(); return test_suite.Run();
} }
}; };
......
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