New issue
Advanced search Search tips

Issue 791791 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 789442
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug



Sign in to add a comment

ServiceProcessControlBrowserTest flaky on Win7 Tests (dbg)(1)

Project Member Reported by imch...@chromium.org, Dec 4 2017

Issue description

The following tests are flaking on Win7 Tests (dbg)(1):

ServiceProcessControlBrowserTest.LaunchAndReconnect
ServiceProcessControlBrowserTest.LaunchAndIPC


Error message:

[ RUN      ] ServiceProcessControlBrowserTest.LaunchAndIPC
[5108:7940:1204/101334.622:WARNING:chrome_browser_main_win.cc(613)] Command line too long for RegisterApplicationRestart:  --brave-new-test-launcher --cfi-diag=0 --gtest_also_run_disabled_tests --gtest_filter=ServiceProcessControlBrowserTest.LaunchAndIPC --single_process --test-launcher-bot-mode --test-launcher-output="C:\Users\CHROME~2\AppData\Local\Temp\scoped_dir5864_25095\results5864_1557\test_results.xml" --test-launcher-summary-output="e:\b\s\w\iotqub6r\output.json" --user-data-dir="C:\Users\CHROME~2\AppData\Local\Temp\scoped_dir5864_25095\d5864_8067" --disable-offline-auto-reload --no-first-run --no-default-browser-check --enable-logging=stderr --disable-default-apps --wm-window-animations-disabled --disable-component-update --test-type=browser --force-color-profile=srgb --disable-zero-browsers-open-for-tests --ipc-connection-timeout=45 --allow-file-access-from-files --dom-automation --log-gpu-control-list-decisions --disable-backgrounding-occluded-windows --disable-gl-drawing-for-tests --override-use-software-gl-for-tests --force-color-profile=srgb --disable-compositor-ukm-for-tests --enable-features=TestFeatureForBrowserTest1 --disable-features=NetworkPrediction,TestFeatureForBrowserTest2 --flag-switches-begin --flag-switches-end --restore-last-session about:blank
[7536:5440:1204/101335.245:INFO:media_foundation_video_encode_accelerator_win.cc(370)] Windows versions earlier than 8 are not supported.
[4620:7608:1204/101337.227:FATAL:post_task.cc(72)] Check failed: TaskScheduler::GetInstance(). Ref. Prerequisite section of post_task.h.
Hint: if this is in a unit test, you're likely merely missing a base::test::ScopedTaskEnvironment member in your fixture.
Backtrace:
	base::debug::StackTrace::StackTrace [0x100BE886+102]
	base::debug::StackTrace::StackTrace [0x100BD423+35]
	logging::LogMessage::~LogMessage [0x10134245+149]
	base::PostDelayedTaskWithTraits [0x102E4305+245]
	base::PostTaskWithTraits [0x102E4766+118]
	??$GetValueFromArgListImpl@U?$BooleanArgGetter@UWithBaseSyncPrimitives@base@@@internal@base@@W4TaskPriority@3@$$V@internal@base@@YA_NUCallSecondTag@01@U?$BooleanArgGetter@UWithBaseSyncPrimitives@base@@@01@ABW4TaskPriority@1@@Z [0x102E56E0+304]
	base::internal::PostTaskAndReplyImpl::PostTaskAndReply [0x103318B3+947]
	base::PostTaskWithTraitsAndReply [0x102E4689+169]
	net::SerialWorker::WorkNow [0x10D0469E+510]
	net::internal::DnsConfigServiceWin::ReadNow [0x10C53003+35]
	net::DnsConfigService::WatchConfig [0x10C4C5C5+661]
	net::NetworkChangeNotifierWin::DnsConfigServiceThread::Init [0x109A46FC+124]
	base::Thread::ThreadMain [0x10363CAE+1294]
	base::PlatformThread::GetCurrentThreadPriority [0x1033125D+1181]
	BaseThreadInitThunk [0x75F9336A+18]
	RtlInitializeExceptionChain [0x77EA9882+99]
	RtlInitializeExceptionChain [0x77EA9855+54]
Received fatal exception EXCEPTION_BREAKPOINT
Backtrace:
	base::debug::BreakDebugger [0x100BB1AD+45]
	??$Bind@A6AXPBDHV?$BasicStringPiece@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@base@@1@Z$$V@base@@YA?AV?$RepeatingCallback@$$A6AXPBDHV?$BasicStringPiece@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@base@@1@Z@0@A6AXPBD [0x06472213+131]
	base::internal::FunctorTraits<void (__cdecl*)(char const *,int,base::BasicStringPiece<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,base::BasicStringPiece<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >),voi [0x064726DA+170]
	base::internal::InvokeHelper<0,void>::MakeItSo<void (__cdecl*const &)(char const *,int,base::BasicStringPiece<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,base::BasicStringPiece<std::basic_string<char,std::char_traits<char>,std:: [0x064725E0+144]
	??$RunImpl@ABQ6AXPBDHV?$BasicStringPiece@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@base@@1@ZABV?$tuple@$$V@std@@$S@?$Invoker@U?$BindState@P6AXPBDHV?$BasicStringPiece@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@base@ [0x06472514+148]
	?Run@?$Invoker@U?$BindState@P6AXPBDHV?$BasicStringPiece@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@base@@1@Z$$V@internal@base@@$$A6AXPBDHV?$BasicStringPiece@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@3@1@Z@internal@ [0x064723A5+165]
	base::RepeatingCallback<void __cdecl(char const *,int,base::BasicStringPiece<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,base::BasicStringPiece<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >)>::Run [0x101367EE+190]
	logging::LogMessage::~LogMessage [0x10134766+1462]
	base::PostDelayedTaskWithTraits [0x102E4305+245]
	base::PostTaskWithTraits [0x102E4766+118]
	??$GetValueFromArgListImpl@U?$BooleanArgGetter@UWithBaseSyncPrimitives@base@@@internal@base@@W4TaskPriority@3@$$V@internal@base@@YA_NUCallSecondTag@01@U?$BooleanArgGetter@UWithBaseSyncPrimitives@base@@@01@ABW4TaskPriority@1@@Z [0x102E56E0+304]
	base::internal::PostTaskAndReplyImpl::PostTaskAndReply [0x103318B3+947]
	base::PostTaskWithTraitsAndReply [0x102E4689+169]
	net::SerialWorker::WorkNow [0x10D0469E+510]
	net::internal::DnsConfigServiceWin::ReadNow [0x10C53003+35]
	net::DnsConfigService::WatchConfig [0x10C4C5C5+661]
	net::NetworkChangeNotifierWin::DnsConfigServiceThread::Init [0x109A46FC+124]
	base::Thread::ThreadMain [0x10363CAE+1294]
	base::PlatformThread::GetCurrentThreadPriority [0x1033125D+1181]
	BaseThreadInitThunk [0x75F9336A+18]
	RtlInitializeExceptionChain [0x77EA9882+99]
	RtlInitializeExceptionChain [0x77EA9855+54]
../../chrome/browser/service_process/service_process_control_browsertest.cc(126): error: Failed
../../chrome/browser/service_process/service_process_control_browsertest.cc(175): error: Value of: ServiceProcessControl::GetInstance()->IsConnected()
  Actual: false
Expected: true
[5108:7940:1204/101338.477:INFO:chrome_cryptauth_service.cc(222)] Profile is not authenticated yet; waiting before starting CryptAuth managers.
[5108:7940:1204/101338.589:WARNING:url_request_context_getter.cc(43)] URLRequestContextGetter leaking due to no owning thread.
[  FAILED  ] ServiceProcessControlBrowserTest.LaunchAndIPC, where TypeParam =  and GetParam() =  (4618 ms)
 
Cc: vitalyb...@chromium.org
Cc: thestig@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 5 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/457e73a60544ef464c92ab89ecd1fdcdab3c6494

commit 457e73a60544ef464c92ab89ecd1fdcdab3c6494
Author: Derek Cheng <imcheng@chromium.org>
Date: Tue Dec 05 00:55:36 2017

Disable 2 flaky ServiceProcessControlBrowserTest tests on Windows.

TBR=vitalybuka@chromium.org

Bug:  791791 
Change-Id: Idd34d0e94153505e4bf151df5d8f0e021adea420
Reviewed-on: https://chromium-review.googlesource.com/807385
Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521560}
[modify] https://crrev.com/457e73a60544ef464c92ab89ecd1fdcdab3c6494/chrome/browser/service_process/service_process_control_browsertest.cc

Cc: rbpotter@chromium.org
Owner: thestig@chromium.org
it's failing almost consistently starting from about https://ci.chromium.org/buildbot/chromium.win/Win7%20Tests%20(dbg)(1)/64945

Looks like regression which needs to be investigated.
Mergedinto: 789442
Status: Duplicate (was: Untriaged)
Project Member

Comment 6 by bugdroid1@chromium.org, Dec 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f85fb155283f6e3af86c157f7cf8bbeb69e6a75d

commit f85fb155283f6e3af86c157f7cf8bbeb69e6a75d
Author: Vitaly Buka <vitalybuka@chromium.org>
Date: Thu Dec 07 23:32:37 2017

Revert "Disable 2 flaky ServiceProcessControlBrowserTest tests on Windows."

Should be fixed by 
https://chromium-review.googlesource.com/810925

This reverts commit 457e73a60544ef464c92ab89ecd1fdcdab3c6494.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> Disable 2 flaky ServiceProcessControlBrowserTest tests on Windows.
> 
> TBR=vitalybuka@chromium.org
> 
> Bug:  791791 
> Change-Id: Idd34d0e94153505e4bf151df5d8f0e021adea420
> Reviewed-on: https://chromium-review.googlesource.com/807385
> Reviewed-by: Derek Cheng <imcheng@chromium.org>
> Commit-Queue: Derek Cheng <imcheng@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521560}

TBR=vitalybuka@chromium.org,imcheng@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug:  791791 
Change-Id: Id13d96a12f527ffb81c7bd7bff97daca683b879f
Reviewed-on: https://chromium-review.googlesource.com/815774
Reviewed-by: Derek Cheng <imcheng@chromium.org>
Reviewed-by: Vitaly Buka <vitalybuka@chromium.org>
Commit-Queue: Vitaly Buka <vitalybuka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522613}
[modify] https://crrev.com/f85fb155283f6e3af86c157f7cf8bbeb69e6a75d/chrome/browser/service_process/service_process_control_browsertest.cc

Sign in to add a comment