Commit 9cd23e86 authored by Xida Chen's avatar Xida Chen Committed by Commit Bot

Removes some deadcode in RenderThreadImplBrowserTest

There is a test that is already removed, the code for disabling the
test is still there. This CL removes the code that disabling the test.

Bug: 728034
Change-Id: I47a6c3e98e8fee64ad776e4ff2c230d0f2733a1f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2488121Reviewed-by: default avatarYi Gu <yigu@chromium.org>
Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#819157}
parent c87be213
...@@ -266,18 +266,6 @@ class RenderThreadImplBrowserTest : public testing::Test, ...@@ -266,18 +266,6 @@ class RenderThreadImplBrowserTest : public testing::Test,
DISALLOW_COPY_AND_ASSIGN(RenderThreadImplBrowserTest); DISALLOW_COPY_AND_ASSIGN(RenderThreadImplBrowserTest);
}; };
// Check that InputHandlerManager outlives compositor thread because it uses
// raw pointers to post tasks.
// Disabled under LeakSanitizer due to memory leaks. http://crbug.com/348994
// Disabled on Windows due to flakiness: http://crbug.com/728034.
#if defined(OS_WIN)
#define MAYBE_InputHandlerManagerDestroyedAfterCompositorThread \
DISABLED_InputHandlerManagerDestroyedAfterCompositorThread
#else
#define MAYBE_InputHandlerManagerDestroyedAfterCompositorThread \
InputHandlerManagerDestroyedAfterCompositorThread
#endif
// Disabled under LeakSanitizer due to memory leaks. // Disabled under LeakSanitizer due to memory leaks.
TEST_F(RenderThreadImplBrowserTest, TEST_F(RenderThreadImplBrowserTest,
WILL_LEAK(NonResourceDispatchIPCTasksDontGoThroughScheduler)) { WILL_LEAK(NonResourceDispatchIPCTasksDontGoThroughScheduler)) {
......
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