Chromium Debug crash, with Task Manager open and Process Priority displayed, while typing URL into omnibox |
|||||
Issue descriptionChrome Version: 57.0.2971.0 OS: Windows What steps will reproduce the problem? (1) Launch Chrome. (2) Open Task Manager. (3) Enable display of Process Priority. (4) Create a new tab in the window. (5) Start typing URL into the new tab. What is the expected result? Expect that you can finish typing in the URL and load the desired content. What happens instead? Chrome fails the DCHECK at https://cs.chromium.org/chromium/src/base/process/process_win.cc?q=Process::IsProcessBackgrounded&sq=package:chromium&l=164 (i.e. the underlying process handle is not valid). IsProcessBackgrounded is being called from TaskGroupSampler::RefreshProcessPriority() in this instance.
,
Jan 5 2017
,
Jan 5 2017
Most likely trigger for this is failure to Open() the process-Id when creating a new TaskGroup - that is inherently racey (e.g. if the target process happens to crash, or exit, immediately after starting, before we reach the Open()) but it is surprising it's so easy to repro this in normal use.
,
Jan 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/76ac066a7ca9091e4b45b459e1d46f413614a1e1 commit 76ac066a7ca9091e4b45b459e1d46f413614a1e1 Author: nick <nick@chromium.org> Date: Fri Jan 20 01:52:56 2017 ShowCreatedWindow: some speculative fixes for 680876. Bug 678399 is an android-only crash in WebContentsImpl::ShowCreatedWindow, that seems to have started with recent changes to CreateNewWindow. The root cause of this issue is not understood, but one possible explanation is that the |pending_contents_| list winds up tracking a dead WebContents. This CL adds some defensive code aimed at either catching a problem earlier, or papering over it. BUG=680876 Review-Url: https://codereview.chromium.org/2646613002 Cr-Commit-Position: refs/heads/master@{#444939} [modify] https://crrev.com/76ac066a7ca9091e4b45b459e1d46f413614a1e1/content/browser/web_contents/web_contents_impl.cc
,
Sep 19
,
Sep 26
,
Nov 21
***UI Mass Triage*** Assuming the issue may be fixed since there is no activity, hence closing.If you feel this issue should still be addressed, please to reopen it or to file a new issue. Thanks! |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by afakhry@chromium.org
, Jan 4 2017Status: Assigned (was: Untriaged)