New issue
Advanced search Search tips

Issue 905461 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Potential jank caused by WebContentsObserverProxy.didFinishNavigation()

Project Member Reported by ssid@google.com, Nov 14

Issue description

We collected slow reports from users facing janks and found that the function mentioned above is taking too much time on the main thread, potentially causing janks.
The issue can be found in reports:
dd57debc83d5c5ba ,2198.09 ms
930844faa4b59f2b ,581.757 ms

Go to crash/ReportID to view the traces.

The stack trace that was hit the most:
nanosleep / lock / malloc
<skipped java stack>
content::WebContentsObserverProxy::DidFinishNavigation(content::NavigationHandle*)
content::WebContentsImpl::DidFinishNavigation(content::NavigationHandle*)
content::NavigationHandleImpl::~NavigationHandleImpl()
content::NavigationHandleImpl::~NavigationHandleImpl()
content::NavigatorImpl::DidNavigate(content::RenderFrameHostImpl*, FrameHostMsg_DidCommitProvisionalLoad_Params const&, std::__ndk1::unique_ptr<content::NavigationHandleImpl, std::__ndk1::default_delete<content::NavigationHandleImpl> >, bool)
content::RenderFrameHostImpl::DidCommitNavigationInternal(FrameHostMsg_DidCommitProvisionalLoad_Params*, bool)
content::RenderFrameHostImpl::DidCommitProvisionalLoad(std::__ndk1::unique_ptr<FrameHostMsg_DidCommitProvisionalLoad_Params, std::__ndk1::default_delete<FrameHostMsg_DidCommitProvisionalLoad_Params> >, mojo::InterfaceRequest<service_manager::mojom::InterfaceProvider>)
content::mojom::FrameHostStubDispatch::Accept(content::mojom::FrameHost*, mojo::Message*)
mojo::InterfaceEndpointClient::HandleIncomingMessageThunk::Accept(mojo::Message*)
_ZN3IPC12_GLOBAL__N_132ChannelAssociatedGroupController19AcceptOnProxyThreadEN4mojo7MessageE$908b0a2b9dcb3d6c3dee58ef8dd98270
base::internal::Invoker<base::internal::BindState<void (IPC::(anonymous namespace)::ChannelAssociatedGroupController::*)(mojo::Message), scoped_refptr<IPC::(anonymous namespace)::ChannelAssociatedGroupController>, base::internal::PassedWrapper<mojo::Message> >, void ()>::Run(base::internal::BindStateBase*)
base::debug::TaskAnnotator::RunTask(char const*, base::PendingTask*)
base::MessageLoop::RunTask(base::PendingTask*)
base::MessageLoop::DoWork()
base::MessagePumpForUI::OnNonDelayedLooperCallback()
base::(anonymous namespace)::NonDelayedLooperCallback(int, int, void*)
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 15

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

commit 57135142b012d42388b24adf4329c5664d50cf14
Author: ssid <ssid@chromium.org>
Date: Thu Nov 15 00:00:03 2018

Add trace event to debug jank issue

BUG=905461

Change-Id: Idee52c3f44ca7aef6707e010b5e07bd668120109
Reviewed-on: https://chromium-review.googlesource.com/c/1330735
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: ssid <ssid@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608179}
[modify] https://crrev.com/57135142b012d42388b24adf4329c5664d50cf14/content/browser/android/web_contents_observer_proxy.cc

Sign in to add a comment