New issue
Advanced search Search tips

Issue 749886 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 749773
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

DCHECK(GetResource()->IsLoaded()) failing in ClassicPendingScript.cpp

Project Member Reported by pdr@chromium.org, Jul 27 2017

Issue description

telemetry perf unittests is failing pretty consistently with this issue.
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_swarming_rel?numbuilds=100
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_swarming_rel/builds/230903
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_swarming_rel/builds/230882

[FATAL:ClassicPendingScript.cpp(172)] Check failed: GetResource()->IsLoaded(). 
Stack Trace:
  	  RELADDR   FUNCTION                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                       FILE:LINE
  	  00000000012fc29f  cc::LayerTreeImpl::ApplySentScrollAndScaleDeltasFromAbortedCommit()                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                            /b/c/b/android_n5x_swarming_rel/src/cc/trees/layer_tree_impl.cc:938
  	  00000000046bd7df  blink::PannerHandler::CalculateDistanceConeGain(blink::FloatPoint3D const&, blink::FloatPoint3D const&, blink::FloatPoint3D const&)                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                            /b/c/b/android_n5x_swarming_rel/src/third_party/WebKit/Source/modules/webaudio/PannerNode.cpp:524
  	  0000000004758097  ChromeContentRendererClient::AddImageContextMenuProperties(blink::WebURLResponse const&, std::__ndk1::map<std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, std::__ndk1::less<std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > >, std::__ndk1::allocator<std::__ndk1::pair<std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > const, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > > > >*)                                            /b/c/b/android_n5x_swarming_rel/src/chrome/renderer/chrome_content_renderer_client.cc:1495
  	  00000000047586af  void service_manager::Connector::BindInterface<safe_browsing::mojom::SafeBrowsing>(std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > const&, mojo::InterfacePtr<safe_browsing::mojom::SafeBrowsing>*)                                                                                                                                                                                                                                                                                                                                                                                                                                                                             /b/c/b/android_n5x_swarming_rel/src/services/service_manager/public/cpp/connector.h:103

This line was recently added in https://chromium.googlesource.com/chromium/src/+/5578c07d877cb8bf3ee41439ea1c8ff9c0dd7394

 

Comment 1 by pdr@chromium.org, Jul 31 2017

This is still failing and is making it hard for me to diagnose another crash in telemetry perf unittests. Can the original patch be reverted?

Comment 2 by jochen@chromium.org, Jul 31 2017

Mergedinto: 749773
Status: Duplicate (was: Assigned)

Sign in to add a comment