New issue
Advanced search Search tips

Issue 765809 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: ----
Type: ----



Sign in to add a comment

PlatformAppBrowserTest.AppWindowRestoreState failing on chromium.win/Win10 Tests x64

Project Member Reported by kelv...@chromium.org, Sep 15 2017

Issue description

renderer_side_navigation_browser_tests failing on chromium.win/Win10 Tests x64

Builders failed on: 
- Win10 Tests x64: 
  https://build.chromium.org/p/chromium.win/builders/Win10%20Tests%20x64


[ RUN      ] PlatformAppBrowserTest.AppWindowRestoreState

[1148:3092:0915/093611.208:INFO:CONSOLE(0)] "[SUCCESS] testRestoreNormal", source: chrome-extension://llcadloajhiiodmppggmekegllofnmlm/_generated_background_page.html (0)

[1148:3092:0915/093611.269:INFO:CONSOLE(0)] "[SUCCESS] testRestoreMaximized", source: chrome-extension://llcadloajhiiodmppggmekegllofnmlm/_generated_background_page.html (0)

Backtrace:

	extensions::ProcessManager::GetExtensionHostForRenderFrameHost [0x00007FF67EDE95A8+312]

	extensions::ProcessManager::GetExtensionHostForRenderFrameHost [0x00007FF67EDE9552+226]

	extensions::ProcessManager::ClearBackgroundPageData [0x00007FF67EDE8491+81]

	extensions::ProcessManager::Observe [0x00007FF67EDE9C2C+332]

	content::NotificationServiceImpl::Notify [0x00007FF67EAE5241+833]

	extensions::ExtensionHost::~ExtensionHost [0x00007FF67EDAA98E+302]

	extensions::ExtensionHost::`vector deleting destructor' [0x00007FF67EDAAC44+20]

	extensions::ProcessManager::CloseBackgroundHosts [0x00007FF67EDE876D+61]

	extensions::ChromeProcessManagerDelegate::OnProfileDestroyed [0x00007FF680D0881E+30]

	content::NotificationServiceImpl::Notify [0x00007FF67EAE5179+633]

	Profile::MaybeSendDestroyedNotification [0x00007FF681341201+65]

	ProfileDestroyer::DestroyProfileWhenAppropriate [0x00007FF68139C364+212]

	ProfileManager::ProfileInfo::~ProfileInfo [0x00007FF681335E45+21]

	std::_Tree<std::_Tmap_traits<base::FilePath,std::unique_ptr<ProfileManager::ProfileInfo,std::default_delete<ProfileManager::ProfileInfo> >,std::less<base::FilePath>,std::allocator<std::pair<base::FilePath const ,std::unique_ptr<ProfileManager::ProfileInfo [0x00007FF67F77AF17+71]

	std::_Tree<std::_Tmap_traits<base::FilePath,std::unique_ptr<ProfileManager::ProfileInfo,std::default_delete<ProfileManager::ProfileInfo> >,std::less<base::FilePath>,std::allocator<std::pair<base::FilePath const ,std::unique_ptr<ProfileManager::ProfileInfo [0x00007FF67F77AF90+48]

	ProfileManager::~ProfileManager [0x00007FF681335F37+199]

	ProfileManager::`vector deleting destructor' [0x00007FF681336004+20]

	BrowserProcessImpl::StartTearDown [0x00007FF6800D49B9+537]

	ChromeBrowserMainParts::PostMainMessageLoopRun [0x00007FF6800C6C8A+346]

	content::BrowserMainLoop::ShutdownThreadsAndCleanUp [0x00007FF67E8EB77E+414]

	content::BrowserMainRunnerImpl::Shutdown [0x00007FF67E8ECDCF+607]

	content::BrowserMain [0x00007FF67E8E6E83+179]

	content::RunNamedProcessTypeMain [0x00007FF67F76E1ED+173]

	content::ContentMainRunnerImpl::Run [0x00007FF67F76E0E6+214]

	service_manager::Main [0x00007FF680771BB8+648]

	content::ContentMain [0x00007FF67F76D73C+44]

	content::BrowserTestBase::SetUp [0x00007FF67F9096C6+1670]

	InProcessBrowserTest::SetUp [0x00007FF67F8431D3+355]

	testing::internal::HandleExceptionsInMethodIfSupported<testing::Test,void> [0x00007FF67E35082D+61]

	testing::Test::Run [0x00007FF67E35BFFA+74]

	testing::TestInfo::Run [0x00007FF67E35C221+161]

	testing::TestCase::Run [0x00007FF67E35C105+165]

	testing::internal::UnitTestImpl::RunAllTests [0x00007FF67E35C5D2+562]

	testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl,bool> [0x00007FF67E3508CD+61]

	testing::UnitTest::Run [0x00007FF67E35C369+217]

	base::TestSuite::Run [0x00007FF67F85FC99+137]

	ChromeTestSuiteRunner::RunTestSuite [0x00007FF68250745F+95]

	content::LaunchTests [0x00007FF67F8FED5A+618]

	LaunchChromeTests [0x00007FF682507233+195]

	main [0x00007FF682506DD8+96]

	__scrt_common_main_seh [0x00007FF6824A6099+285] (f:\dd\vctools\crt\vcstartup\src\startup\exe_common.inl:253)

	BaseThreadInitThunk [0x00007FF991278364+20]

	RtlUserThreadStart [0x00007FF993B75E91+33]

[621/623] PlatformAppBrowserTest.AppWindowRestoreState (2847 ms)
[622/623] StartupBrowserCreatorCorruptProfileTest.PRE_DoNotStartLockedProfile (2045 ms)
[623/623] StartupBrowserCreatorCorruptProfileTest.DoNotStartLockedProfile (834 ms)
Retrying 1 test (retry #2)

 
stevejb@, I assigned the bug to you based on the TODO statement on the browser test.  Please feel free to assign to the correct owner if appropriate.
Cc: steve...@chromium.org benwells@chromium.org rdevlin....@chromium.org
Components: Platform>Apps>API>Windows
Owner: ----
Status: Untriaged (was: Assigned)
What TODO is that?

I haven't been in this code in a very long time. Unassigning for triage.

Project Member

Comment 3 by bugdroid1@chromium.org, Sep 15 2017

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

commit 81df73d695a0a64edfeee6ed09d82b94a22b6eae
Author: kelvinp@chromium.org <kelvinp@chromium.org>
Date: Fri Sep 15 22:13:39 2017

Disable PlatformAppBrowserTest.AppWindowRestoreState on Windows

Tbr: benwells@chromium.org
No-Try: true
Bug: 765809
Change-Id: I3121c1c2dc2e0293f045cbb1c89e9bc1a553eecc
Reviewed-on: https://chromium-review.googlesource.com/669603
Commit-Queue: Kelvin Poon <kelvinp@chromium.org>
Reviewed-by: Kelvin Poon <kelvinp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502399}
[modify] https://crrev.com/81df73d695a0a64edfeee6ed09d82b94a22b6eae/chrome/browser/apps/app_browsertest.cc

Labels: -Sheriff-Chromium
-Sheriff-Chromium as the test is disabled.
Project Member

Comment 6 by bugdroid1@chromium.org, Sep 18 2017

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

commit 3cb3b8de1e3fa301ac08013a389912bc1e9e9436
Author: Max Morin <maxmorin@chromium.org>
Date: Mon Sep 18 12:35:58 2017

Disable PlatformAppBrowserTest.AppWindowRestoreState

on Mac.

Bug: 765809
Change-Id: I63a92cb682f757a6c13d87f40dddb26171143e15
TBR: benwells@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/670606
Reviewed-by: Max Morin <maxmorin@chromium.org>
Commit-Queue: Max Morin <maxmorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502560}
[modify] https://crrev.com/3cb3b8de1e3fa301ac08013a389912bc1e9e9436/chrome/browser/apps/app_browsertest.cc

Sign in to add a comment