New issue
Advanced search Search tips

Issue 853790 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug
Proj-VR
Proj-XR
Proj-XR-VR

Blocking:
issue 801034



Sign in to add a comment

GPU FYI Win10 FYI Release (Intel HD 630) vr_browser_test crash flake

Project Member Reported by weiliangc@chromium.org, Jun 18 2018

Issue description

https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win10%20FYI%20Release%20%28Intel%20HD%20630%29?limit=200

LOG:
[1/16] VrBrowserTestStandard.DISABLED_TestPresentationLocksFocus (10632 ms)
[ RUN      ] VrBrowserTestStandard.DISABLED_TestPresentationPixels
[1412:7640:0615/175750.343:WARNING:discovery_network_list_win.cc(195)] Failed to open Wlan client handle: 1062
[1412:4148:0615/175750.640:WARNING:chrome_browser_main_win.cc(653)] Command line too long for RegisterApplicationRestart:  --brave-new-test-launcher --cfi-diag=0 --enable-gpu --enable-pixel-output-in-tests --gtest_also_run_disabled_tests --gtest_filter=VrBrowserTestStandard.DISABLED_TestPresentationPixels --single_process --test-launcher-bot-mode --test-launcher-jobs=1 --test-launcher-output="C:\Users\CHROME~1\AppData\Local\Temp\scoped_dir1944_12464\results1944_28836\test_results.xml" --test-launcher-summary-output="c:\b\s\w\iovv4ifv\output.json" --user-data-dir="C:\Users\CHROME~1\AppData\Local\Temp\scoped_dir1944_12464\d1944_24777" --enable-webvr --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 --force-color-profile=srgb --disable-compositor-ukm-for-tests --enable-features=GamepadExtensions,OpenVR,TestFeatureForBrowserTest1 --disable-features=NetworkPrediction,SpeculativePreconnect,TestFeatureForBrowserTest2 --flag-switches-begin --flag-switches-end --restore-last-session about:blank
[1412:4148:0615/175751.726:INFO:CONSOLE(56)] "Test result: Pass", source: file:///C:/b/s/w/ir/chrome/test/data/vr/e2e_test_files/resources/webvr_e2e.js (56)
[1412:4148:0615/175751.726:INFO:CONSOLE(57)] "Test result string: PASS Pixel data is correct. ", source: file:///C:/b/s/w/ir/chrome/test/data/vr/e2e_test_files/resources/webvr_e2e.js (57)
Received fatal exception EXCEPTION_ACCESS_VIOLATION
Backtrace:
	(No symbol) [0x6A3B41A0]
	(No symbol) [0x6A3B174E]
	device::OpenVRRenderLoop::SubmitFrameWithTextureHandle [0x052D8C2F+323]
	device::mojom::VRPresentationProviderStubDispatch::Accept [0x0399C195+1089]
	device::mojom::VRPresentationProviderStub<mojo::RawPtrImplRefTraits<device::mojom::VRPresentationProvider> >::Accept [0x052D9963+19]
	mojo::InterfaceEndpointClient::HandleValidatedMessage [0x03562159+541]
	mojo::FilterChain::Accept [0x0441F4F3+131]
	mojo::InterfaceEndpointClient::HandleIncomingMessage [0x03563006+106]
	mojo::internal::MultiplexRouter::ProcessIncomingMessage [0x0356658A+698]
	mojo::internal::MultiplexRouter::Accept [0x03566111+295]
	mojo::FilterChain::Accept [0x0441F4F3+131]
	mojo::Connector::ReadSingleMessage [0x03560830+364]
	mojo::Connector::ReadAllAvailableMessages [0x03560F0D+87]
	mojo::Connector::OnHandleReadyInternal [0x03560DDC+126]
	base::internal::Invoker<base::internal::BindState<void (__thiscall base::internal::AdaptCallbackForRepeatingHelper<SkBitmap const &>::*)(SkBitmap const &),std::unique_ptr<base::internal::AdaptCallbackForRepeatingHelper<SkBitmap const &>,std::default_delet [0x0088D38F+15]
	mojo::SimpleWatcher::DiscardReadyState [0x01CB1E46+24]
	base::internal::Invoker<base::internal::BindState<net::URLRequestJob * (__cdecl*)(base::FilePath const &,net::URLRequest *,net::NetworkDelegate *),base::FilePath>,net::URLRequestJob * __cdecl(net::URLRequest *,net::NetworkDelegate *)>::Run [0x008FF891+19]
	mojo::SimpleWatcher::OnHandleReady [0x0326F53A+224]
	base::internal::Invoker<base::internal::BindState<void (__thiscall mojo::SimpleWatcher::*)(int,unsigned int,mojo::HandleSignalsState const &),base::WeakPtr<mojo::SimpleWatcher>,int,unsigned int,mojo::HandleSignalsState>,void __cdecl(void)>::Run [0x0326F7B4+58]
	base::debug::TaskAnnotator::RunTask [0x0315A8F4+308]
	base::internal::IncomingTaskQueue::RunTask [0x040C5E4C+108]
	base::MessageLoop::RunTask [0x0317D967+519]
	base::MessageLoop::DeferOrRunPendingTask [0x0317DCBD+157]
	base::MessageLoop::DoWork [0x0317DEDA+506]
	base::MessagePumpDefault::Run [0x040C868C+156]
	base::MessageLoop::Run [0x0317D484+116]
	base::RunLoop::Run [0x031A2C7C+204]
	base::Thread::Run [0x031BDA94+164]
	base::Thread::ThreadMain [0x031BDD1D+621]
	base::PlatformThread::GetCurrentThreadPriority [0x031BBDF5+469]
	BaseThreadInitThunk [0x770A8674+36]
	RtlGetAppContainerNamedObjectPath [0x772D4B47+311]
	RtlGetAppContainerNamedObjectPath [0x772D4B17+263]
[2/16] VrBrowserTestStandard.DISABLED_TestPresentationPixels (CRASHED)


First time flakes is https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win10%20FYI%20Release%20%28Intel%20HD%20630%29/974

Suspect: https://chromium-review.googlesource.com/c/chromium/src/+/1100157

 
Labels: -Pri-2 Pri-1
This is affecting the ANGLE CQ pretty badly. We are having trouble landing CLs. Going to try reverting this.
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 18 2018

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

commit c2f09bb4697acd995d3e22fc22cc99f751c6c313
Author: Jamie Madill <jmadill@chromium.org>
Date: Mon Jun 18 18:51:32 2018

Revert "Add an interface to communicate between the fake openvr and tests"

This reverts commit edb91802453505dabf855c2025df79b0a586a425.

Reason for revert: Suspecting flaky crashes:
Win10 FYI Release (Intel HD 630) vr_browser_test

https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win10%20FYI%20Release%20%28Intel%20HD%20630%29/974

Received fatal exception EXCEPTION_ACCESS_VIOLATION
Backtrace:
	(No symbol) [0x6A3B41A0]
	(No symbol) [0x6A3B174E]
	device::OpenVRRenderLoop::SubmitFrameWithTextureHandle [0x052D8C2F+323]
	device::mojom::VRPresentationProviderStubDispatch::Accept [0x0399C195+1089]
	device::mojom::VRPresentationProviderStub<mojo::RawPtrImplRefTraits<device::mojom::VRPresentationProvider> >::Accept [0x052D9963+19]
	mojo::InterfaceEndpointClient::HandleValidatedMessage [0x03562159+541]
	mojo::FilterChain::Accept [0x0441F4F3+131]
	mojo::InterfaceEndpointClient::HandleIncomingMessage [0x03563006+106]
	mojo::internal::MultiplexRouter::ProcessIncomingMessage [0x0356658A+698]
	mojo::internal::MultiplexRouter::Accept [0x03566111+295]
	mojo::FilterChain::Accept [0x0441F4F3+131]
	mojo::Connector::ReadSingleMessage [0x03560830+364]
	mojo::Connector::ReadAllAvailableMessages [0x03560F0D+87]
<snip>

Bug:  853790 

Original change's description:
> Add an interface to communicate between the fake openvr and tests
> 
> This change creates a way for tests to control the behavior of
> the fake OpenVR, and allows devices to send information up to
> tests.
> 
> BUG= 801034 
> 
> Change-Id: Ie0fe035ada9553114c054717f52788d375d5c62f
> Reviewed-on: https://chromium-review.googlesource.com/1100157
> Commit-Queue: Bill Orr <billorr@chromium.org>
> Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#567725}

TBR=jam@chromium.org,bsheedy@chromium.org,billorr@chromium.org

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

Bug:  801034 
Change-Id: Ie55238c46b1d9c34d92bee2b6324539912b47846
Reviewed-on: https://chromium-review.googlesource.com/1104837
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Bill Orr <billorr@chromium.org>
Commit-Queue: Jamie Madill <jmadill@chromium.org>
Cr-Commit-Position: refs/heads/master@{#568093}
[modify] https://crrev.com/c2f09bb4697acd995d3e22fc22cc99f751c6c313/chrome/browser/vr/test/DEPS
[delete] https://crrev.com/bfad34f4f4251c94ae82b8b602d293968c9927f9/chrome/browser/vr/test/mock_openvr_device_hook_base.cc
[delete] https://crrev.com/bfad34f4f4251c94ae82b8b602d293968c9927f9/chrome/browser/vr/test/mock_openvr_device_hook_base.h
[modify] https://crrev.com/c2f09bb4697acd995d3e22fc22cc99f751c6c313/chrome/browser/vr/test/vr_xr_browser_test.cc
[modify] https://crrev.com/c2f09bb4697acd995d3e22fc22cc99f751c6c313/chrome/browser/vr/webvr_pixel_browser_test.cc
[modify] https://crrev.com/c2f09bb4697acd995d3e22fc22cc99f751c6c313/chrome/test/BUILD.gn
[modify] https://crrev.com/c2f09bb4697acd995d3e22fc22cc99f751c6c313/device/vr/BUILD.gn
[modify] https://crrev.com/c2f09bb4697acd995d3e22fc22cc99f751c6c313/device/vr/openvr/openvr_device_provider.cc
[modify] https://crrev.com/c2f09bb4697acd995d3e22fc22cc99f751c6c313/device/vr/openvr/openvr_device_provider.h
[modify] https://crrev.com/c2f09bb4697acd995d3e22fc22cc99f751c6c313/device/vr/openvr/test/fake_openvr_impl_api.cc
[add] https://crrev.com/c2f09bb4697acd995d3e22fc22cc99f751c6c313/device/vr/openvr/test/fake_openvr_log.h
[delete] https://crrev.com/bfad34f4f4251c94ae82b8b602d293968c9927f9/device/vr/openvr/test/test_helper.cc
[delete] https://crrev.com/bfad34f4f4251c94ae82b8b602d293968c9927f9/device/vr/openvr/test/test_helper.h
[delete] https://crrev.com/bfad34f4f4251c94ae82b8b602d293968c9927f9/device/vr/openvr/test/test_hook.h

Status: Fixed (was: Assigned)
So far looks stable on the GPU bots. Thanks Bill!

Comment 4 by kbr@chromium.org, Jun 20 2018

Blocking: 801034

Sign in to add a comment