Commit 463bf383 authored by Xida Chen's avatar Xida Chen Committed by Commit Bot

Disable DeleteActiveSession on Win

In my previous attemp to disable this here:
https://chromium-review.googlesource.com/c/chromium/src/+/1026155, there
was a typo and causes compile failure. I have a full build with this CL
and it works fine.

TBR=thestig@chromium.org
NOTRY=true

Bug: 836289
Change-Id: I92df34158b417b87adf48e9b8460d10171594a3e
Reviewed-on: https://chromium-review.googlesource.com/1024603
Commit-Queue: Xida Chen <xidachen@chromium.org>
Reviewed-by: default avatarXida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553209}
parent 46eca389
......@@ -192,7 +192,13 @@ IN_PROC_BROWSER_TEST_P(TwoClientSessionsSyncTest, DeleteIdleSession) {
EXPECT_FALSE(GetSessionData(1, &sessions1));
}
IN_PROC_BROWSER_TEST_P(TwoClientSessionsSyncTest, DeleteActiveSession) {
// Timeout on win (https://crbug.com/836289).
#if defined(OS_WIN)
#define MAYBE_DeleteActiveSession DISABLED_DeleteActiveSession
#else
#define MAYBE_DeleteActiveSession DeleteActiveSession
#endif
IN_PROC_BROWSER_TEST_P(TwoClientSessionsSyncTest, MAYBE_DeleteActiveSession) {
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
ASSERT_TRUE(CheckInitialState(0));
......
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