New issue
Advanced search Search tips

Issue 698889 link

Starred by 0 users
Status: Archived
Owner:
xiy...@chromium.org
Closed: Mar 2017
Cc:
mustash-bugs@google.com
derat@chromium.org
jdufault@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Mash login crashes after login due to schedule lock screen preload using UserActivityDetector

Project Member Reported by xiy...@chromium.org, Mar 6 2017 
The preload webui screen lock code use IdleDetector -> UserActivityDetector, that is not created as part of brwoser. Hence the crash.

I will disable the preload for now and revisit this after issue 626899.
====

Received signal 11 SEGV_MAPERR 000000000018
#0 0x7f9f0d1b36cb base::debug::StackTrace::StackTrace()
#1 0x7f9f0d1b1d9c base::debug::StackTrace::StackTrace()
#2 0x7f9f0d1b31df base::debug::(anonymous namespace)::StackDumpSignalHandler()
#3 0x7f9f0d5f7330 <unknown>
#4 0x7f9f0a400dec std::__cxx1998::vector<>::size()
#5 0x7f9f0a43fb58 base::ObserverListBase<>::HasObserver()
#6 0x7f9f0a43f18c ui::UserActivityDetector::HasObserver()
#7 0x7f9f0f05b19e chromeos::IdleDetector::Start()
#8 0x7f9f0f1a1b34 chromeos::PreloadedWebView::PreloadOnIdle()
#9 0x7f9f0f1052ba chromeos::WebUIScreenLocker::RequestPreload()
#10 0x7f9f0f14888a chromeos::ChromeSessionManager::SessionStarted()
#11 0x7f9f0f14b9f7 chromeos::UserSessionManager::DoBrowserLaunchInternal()
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 7 2017 

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

commit 3bdf8977c45b8073999d40f2f7ec0f56dce7aec4
Author: xiyuan <xiyuan@chromium.org>
Date: Tue Mar 07 17:16:59 2017

mash: Fix screen lock preload crash after login

Disable screen lock preload in mash to avoid the crash for now.

BUG= 698889 

Review-Url: https://codereview.chromium.org/2736813002
Cr-Commit-Position: refs/heads/master@{#455108}

[modify] https://crrev.com/3bdf8977c45b8073999d40f2f7ec0f56dce7aec4/chrome/browser/chromeos/login/lock/webui_screen_locker.cc

Comment 2 by xiy...@chromium.org, Mar 7 2017

Status: Fixed (was: Assigned)
Close this one and filed  issue 699144  to re-enable the preloading when IdleDetector -> UserActivityDetector works on mash.

Comment 3 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 4 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 5 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 6 by lafo...@chromium.org, Feb 26 2018

Components: -Internals>MUS Internals>Services>WindowService

Comment 7 by lafo...@chromium.org, Feb 26 2018

Components: -MUS
►

Sign in to add a comment