NOTREACHED in StandardManagementPolicyProvider::GetDebugPolicyProviderName |
||
Issue descriptionChrome Version: 68.0.3438 dev build git rev 18432a5b72da87c0f47 OS: Windows 10 (gWindows) What steps will reproduce the problem? (1) Compile chrome with args below [1] (2) Run Chrome --enable-logging --v=0 (3) What is the expected result? No NOTREACHED is hit What happens instead? NOTREACHED is hit NOTREACHED here is being hit: https://cs.chromium.org/chromium/src/chrome/browser/extensions/standard_management_policy_provider.cc?l=68 std::string StandardManagementPolicyProvider::GetDebugPolicyProviderName() const { #ifdef NDEBUG NOTREACHED(); <--- HERE return std::string(); #else return "extension management policy controlled settings"; #endif } This is being called from ManagementPolicy::ApplyToProviderList DVLOG(1) << debug_operation_name << " of extension " << name << " (" << id << ")" << " prohibited by " << provider->GetDebugPolicyProviderName(); debug_operation_name is "Disabling" name is "Chrome Web Store Payments" id is "nmmhkkegccagdldgiimedpiccmgmieda" Possibly because this is a release with dchecks enabled, and so not expecting for DVLOG(1) to be called in release? Suggest changing the #if defined(NDEBUG) to #if defined(NDEBUG) && !defined(DCHECK_ALWAYS_ON) [1] # Build arguments go here. # See "gn args <out_dir> --list" for available build arguments. use_goma = true goma_dir = "C:\src\goma" is_component_build = true is_debug = false target_cpu = "x64" enable_nacl = false remove_webcore_debug_symbols = true is_chrome_branded = true is_clang = true dcheck_always_on = true
,
May 29 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/72a97a3e5a3f74bbb82b12f3a27d013f449c5f0b commit 72a97a3e5a3f74bbb82b12f3a27d013f449c5f0b Author: Will Harris <wfh@chromium.org> Date: Tue May 29 19:05:21 2018 Fix GetDebugPolicyProviderName to handle DCHECK_ALWAYS_ON BUG= 846436 Change-Id: I9b3b504c75660d9d65fe24bbf0c592915f9f6b19 Reviewed-on: https://chromium-review.googlesource.com/1072508 Commit-Queue: Will Harris <wfh@chromium.org> Reviewed-by: Wez <wez@chromium.org> Reviewed-by: Devlin <rdevlin.cronin@chromium.org> Reviewed-by: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#562536} [modify] https://crrev.com/72a97a3e5a3f74bbb82b12f3a27d013f449c5f0b/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc [modify] https://crrev.com/72a97a3e5a3f74bbb82b12f3a27d013f449c5f0b/chrome/browser/chromeos/extensions/signin_screen_policy_provider.cc [modify] https://crrev.com/72a97a3e5a3f74bbb82b12f3a27d013f449c5f0b/chrome/browser/extensions/standard_management_policy_provider.cc [modify] https://crrev.com/72a97a3e5a3f74bbb82b12f3a27d013f449c5f0b/chrome/browser/supervised_user/supervised_user_service.cc [modify] https://crrev.com/72a97a3e5a3f74bbb82b12f3a27d013f449c5f0b/chrome/browser/supervised_user/supervised_user_service_unittest.cc [modify] https://crrev.com/72a97a3e5a3f74bbb82b12f3a27d013f449c5f0b/extensions/browser/management_policy.h
,
May 29 2018
This should be Fixed. |
||
►
Sign in to add a comment |
||
Comment 1 by wfh@chromium.org
, May 24 2018Owner: wfh@chromium.org
Status: Started (was: Untriaged)