New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 815246 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Long OOO (go/where-is-mgiuca)
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug

Blocked on:
issue 814400



Sign in to add a comment

HostedAppTest.ShouldShowLocationBarForHTTPSApp flaky crashes on Windows

Project Member Reported by reillyg@chromium.org, Feb 23 2018

Issue description

Dashboard:

https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=HostedAppTest.ShouldShowLocationBarForHTTPSApp

Example crash:

https://chromium-swarm.appspot.com/task?id=3bdc5de34d977010

[ RUN      ] HostedAppTest.ShouldShowLocationBarForHTTPSApp/1
[5568:1724:0223/085223.014:WARNING:discovery_network_list_win.cc(195)] Failed to open Wlan client handle: 1062
[5568:6376:0223/085223.019:WARNING:chrome_browser_main_win.cc(615)] Command line too long for RegisterApplicationRestart:  --brave-new-test-launcher --cfi-diag=0 --gtest_also_run_disabled_tests --gtest_filter=HostedAppTest.ShouldShowLocationBarForHTTPSApp/1 --single_process --test-launcher-bot-mode --test-launcher-output="C:\Users\CHROME~2\AppData\Local\Temp\scoped_dir7648_18665\results7648_2505\test_results.xml" --test-launcher-summary-output="e:\b\swarm_slave\w\ioyo7ra_\output.json" --user-data-dir="C:\Users\CHROME~2\AppData\Local\Temp\scoped_dir7648_18665\d7648_11051" --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=30 --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=DesktopPWAWindowing,TestFeatureForBrowserTest1 --disable-features=NetworkPrediction,TestFeatureForBrowserTest2 --flag-switches-begin --flag-switches-end --restore-last-session about:blank
[5568:6376:0223/085226.059:INFO:chrome_cryptauth_service.cc(232)] Profile is not authenticated yet; waiting before starting CryptAuth managers.
Received fatal exception EXCEPTION_ACCESS_VIOLATION
Backtrace:
	network::ResourceScheduler::OnClientDeleted [0x0388AE69+27]
	network::ResourceSchedulerClient::~ResourceSchedulerClient [0x03893C05+19]
	scoped_refptr<network::ResourceSchedulerClient>::~scoped_refptr<network::ResourceSchedulerClient> [0x0388E43D+21]
	network::URLLoader::~URLLoader [0x0389983D+163]
	network::URLLoader::`scalar deleting destructor' [0x0389902B+11]
	ChromeOmniboxNavigationObserver::WebContentsDestroyed [0x054FBA9D+13]
	net::SSLConfigService::NotifySSLConfigChange [0x02A1AC78+86]
	ProfileIOData::ShutdownOnUIThread [0x0420D63B+457]
	base::internal::FunctorTraits<void (__cdecl*)(std::unique_ptr<wifi::WiFiService,std::default_delete<wifi::WiFiService> >),void>::Invoke<std::unique_ptr<wifi::WiFiService,std::default_delete<wifi::WiFiService> > > [0x03DC674C+34]
	base::internal::Invoker<base::internal::BindState<void (__cdecl*)(std::unique_ptr<net::URLRequestInterceptor,std::default_delete<net::URLRequestInterceptor> >),std::unique_ptr<net::URLRequestInterceptor,std::default_delete<net::URLRequestInterceptor> > >, [0x04DCD312+18]
	base::debug::TaskAnnotator::RunTask [0x0292CD09+153]
	base::internal::IncomingTaskQueue::RunTask [0x0294DE73+19]
	base::MessageLoop::RunTask [0x028E9426+438]
	base::MessageLoop::DeferOrRunPendingTask [0x028E9763+83]
	base::MessageLoop::DoWork [0x028E9873+211]
	base::MessagePumpForIO::DoRunLoop [0x0295017D+285]
	base::MessagePumpWin::Run [0x0294F77E+110]
	base::MessageLoop::Run [0x028E917F+31]
	base::RunLoop::Run [0x028C7D2E+46]
	base::Thread::Run [0x028FC49B+11]
	content::BrowserThreadImpl::IOThreadRun [0x01BAB4B2+44]
	content::BrowserThreadImpl::Run [0x01BAB5E0+192]
	base::Thread::ThreadMain [0x028FC605+341]
	base::PlatformThread::GetCurrentThreadPriority [0x028E187B+299]
	BaseThreadInitThunk [0x76E4337A+18]
	RtlInitializeExceptionChain [0x775E9882+99]
	RtlInitializeExceptionChain [0x775E9855+54]
[847/927] HostedAppTest.ShouldShowLocationBarForHTTPSApp/1 (CRASHED)

 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 23 2018

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

commit 8536f5b4c49ef6be78ee64477e59ca9b1ecdd9e0
Author: Reilly Grant <reillyg@chromium.org>
Date: Fri Feb 23 19:09:43 2018

Disable flaky HostedAppTests

ShouldShowLocationBarForHTTPApp and ShouldShowLocationBarForHTTPSApp
are both flaky, with the former failing while the later crashes.

TBR=mguica@chromium.org

Bug:  815240 , 815246
Change-Id: I8856047c78d503affea58a25ba6f708454642116
Reviewed-on: https://chromium-review.googlesource.com/934926
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#538848}
[modify] https://crrev.com/8536f5b4c49ef6be78ee64477e59ca9b1ecdd9e0/chrome/browser/ui/extensions/hosted_app_browsertest.cc

Comment 2 by mgiuca@chromium.org, Feb 28 2018

Blockedon: 814400
Owner: mgiuca@chromium.org
Status: Assigned (was: Untriaged)
Similar to Issue 814400. Possibly same cause.
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 1 2018

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

commit 46e6869492fde25b773d9782dfbc48bbd834ad8d
Author: Matt Giuca <mgiuca@chromium.org>
Date: Thu Mar 01 02:58:26 2018

Update to disablement of HostedAppTest.ShouldShowLocationBarForHTTPApp/*

Only disable on Win and macOS (not flaky on other platforms), and update
comments.

Bug: 814400, 815246
Tbr: ortuno@chromium.org
Change-Id: If31ac6e85a346d5938071b5fcd8862d1bf21081a
Reviewed-on: https://chromium-review.googlesource.com/940643
Commit-Queue: Matt Giuca <mgiuca@chromium.org>
Reviewed-by: Matt Giuca <mgiuca@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540006}
[modify] https://crrev.com/46e6869492fde25b773d9782dfbc48bbd834ad8d/chrome/browser/ui/extensions/hosted_app_browsertest.cc

Sign in to add a comment