New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 893116 link

Starred by 2 users

Issue metadata

Status: Unconfirmed
Owner:
Last visit 28 days ago
Cc:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Using DevTools from non default profile leads to DCHECK AssertBlockingAllowed

Project Member Reported by s...@yandex-team.ru, Oct 8

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:62.0) Gecko/20100101 Firefox/62.0

Steps to reproduce the problem:
1. Create profile 1
2. Switch to profile 1
3. Close default profile (profile with folder 'Default').
4. Close browser and open again.
5. Browser has only 1 active profile. It is 'profile 1'
6. Open any webpage and open dev tools on that page.
7. Get assert because DevTools tries to load 'Default' profile from UI thread.

The stack will be looked like this:
base.dll!base::debug::BreakDebugger() Line 27	C++
base.dll!logging::LogMessage::~LogMessage() Line 886	C++
base.dll!base::AssertBlockingAllowed() Line 35	C++
base.dll!base::PathExists(const base::FilePath & path) Line 453	C++
browser.dll!Profile::CreateProfile(const base::FilePath & path, Profile::Delegate * delegate, Profile::CreateMode create_mode) Line 517	C++
browser.dll!ProfileManager::CreateProfileHelper(const base::FilePath & path) Line 1449	C++
browser.dll!ProfileManager::CreateAndInitializeProfile(const base::FilePath & profile_dir) Line 1525	C++
browser.dll!ProfileManager::GetProfile(const base::FilePath & profile_dir) Line 549	C++
browser.dll!ProfileManager::GetActiveUserOrOffTheRecordProfileFromPath(const base::FilePath & user_data_dir) Line 1491	C++
browser.dll!ProfileManager::GetActiveUserProfile() Line 517	C++
browser.dll!DevToolsBrowserContextManager::GetDefaultBrowserContext() Line 56	C++
browser.dll!ChromeDevToolsManagerDelegate::GetDefaultBrowserContext() Line 231	C++
browser.dll!ChromeDevToolsManagerDelegate::UpdateDeviceDiscovery() Line 297	C++

What is the expected behavior?
No any assert from AssertBlockingAllowed

What went wrong?
Recieve DCHECK

Crashed report ID: No 

How much crashed? Whole browser

Is it a problem with a plugin? No 

Did this work before? N/A 

Chrome version: 70.0.3538.5  Channel: n/a
OS Version: 10.0
Flash Version: Shockwave Flash 31.0 r0

I suppose the problem is from this commit https://chromium.googlesource.com/chromium/src/+/36299bc62c0d5d99c6c3bfecf9db2495d9d2e666
 
Cc: lushnikov@chromium.org
Components: UI>Browser>Profiles
Cc: bsazonov@chromium.org
Owner: lushnikov@chromium.org
At first glance it looks like DevToolsBrowserContextManager::GetDefaultBrowserContext should be using GetLastUsedProfile instead of GetActiveUserProfile. Reassigning to Andrey to validate this.
Project Member

Comment 4 by sheriffbot@chromium.org, Nov 9

Cc: droger@chromium.org msarda@chromium.org ew...@chromium.org tangltom@chromium.org sabineb@chromium.org jlebel@chromium.org
Status: WontFix (was: Unconfirmed)
--Chrome Identity automated triaging--

This bug is Unconfirmed and has gone two weeks without any activity, so it is being closed as WontFix. Please re-open if this is still a valid and reproducible bug or feature request and mark it as Available. Please see https://goo.gl/78kbny for more details. Please remove the Services>SignIn or UI>Browser>Profiles components if this bug isn't related to Chrome Identity.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Components: -UI>Browser>Profiles
Status: Unconfirmed (was: WontFix)
Removing the Profiles component, so it doesn't get automatically closed out, and re-opening.

Sign in to add a comment