signin_screen_policy_provider.cc(61) spam running local cros builds. |
|||||||
Issue descriptionsignin_screen_policy_provider.cc(61) spam running local cros builds. Running a ToT (@#438603) local chrome os build yields a fair amount of spam (see below). Perhaps this belongs at a higher logging level, like DVLOG(2)? [5605:5605:1214/151916.584837:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : ahfgeienlihckogmohjhadlkjgocpleb / Web Store because of 'signin_screen' is only allowed for packaged apps, but this is a hosted app. [5605:5605:1214/151916.585082:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : cjbfomnbifhcdnihkgipgfcihmgjfhbf / Audio Player because of 'signin_screen' is not allowed for specified install location. [5605:5605:1214/151916.585276:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : cnbgggchhmkkdmeppjobngjoejnihlei / Play Store because of 'signin_screen' is not allowed for specified install location. [5605:5605:1214/151916.585458:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : eemcgdkfndhakfknompkggombfjjjeno / Bookmark Manager because of 'signin_screen' is only allowed for packaged apps, but this is a extension. [5605:5605:1214/151916.585651:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : fbjakikfhfdajcamjleinfciajelkpek / Chrome Web Store Gallery because of 'signin_screen' is not allowed for specified install location. [5605:5605:1214/151916.585840:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : hhaomjibdihmijegdhdafkllkbggdgoj / Files because of 'signin_screen' is not allowed for specified install location. [5605:5605:1214/151916.586129:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : idddmepepmjcgiedknnmlbadcokidhoa / NCDLauncher because of 'signin_screen' is only allowed for packaged apps, but this is a extension. [5605:5605:1214/151916.586324:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : jcgeabjmjgoblfofpppfkcoakmfobdko / Video Player because of 'signin_screen' is not allowed for specified install location. [5605:5605:1214/151916.586516:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : jdgcneonijmofocbhmijhacgchbihela / Welcome because of 'signin_screen' is not allowed for specified install location. [5605:5605:1214/151916.586703:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : kddnkjkcjddckihglkfcickdhbmaodcn / Chrome Goodies because of 'signin_screen' is only allowed for packaged apps, but this is a extension. [5605:5605:1214/151916.586992:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : kodldpbjkkmmnilagfdheibampofhaom / Connectivity Diagnostics because of 'signin_screen' is not allowed for specified install location. [5605:5605:1214/151916.587176:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : mgndgikekgjfcpckkfioiadnlibdjbkf / Chromium because of 'signin_screen' is only allowed for packaged apps, but this is a hosted app. [5605:5605:1214/151916.587361:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : mhjfbmdgcfjbbpaeojofohoefgiehjai / Chromium PDF Viewer because of 'signin_screen' is only allowed for packaged apps, but this is a extension. [5605:5605:1214/151916.587644:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : nkoccljplnhpfnfiajclkommnmllphnl / crosh_builtin because of 'signin_screen' is only allowed for packaged apps, but this is a extension. [5605:5605:1214/151916.587834:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : nlkncpkkdoccmpiclbokaimcnedabhhm / Gallery because of 'signin_screen' is not allowed for specified install location. [5605:5605:1214/151916.588023:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : obklkkbkpaoaejdabbfldmcfplpdgolj / Wallpaper Picker because of 'signin_screen' is not allowed for specified install location. [5605:5605:1214/151916.588206:WARNING:signin_screen_policy_provider.cc(61)] Denying load of Extension : pmfjbimdmchhbnneeidfognadeopoehp / Image loader because of 'signin_screen' is only allowed for packaged apps, but this is a extension.
,
Dec 15 2016
We want this output, as it would be the only feedback the extension developer can get for problems with login extension installation - we can not put it to (D)VLOG, only to usual logs. It seems that you're running without --login-manager flag? If you run without --login-manager on dev build, chrome uses login profile (profile specified in --user-data-dir, without user-name-based subdirectory) for user session, and extra policy provider (that runs only for login profile) is started.
,
Dec 15 2016
Okay, thanks for letting me know that it's intentional; feel free to Won'tFix, I guess. Yeah, I generally run chrome without flags, if possible.
,
Dec 23 2016
Denis, isn't it a bug to apply all these login screen checks to the profile used for the normal user session? I think the intention here was to put these restrictions to the Chrome OS login screen only (which runs locally only if there is --login-manager command line flag specified). It doesn't make sense to block loading of some extensions in the normal user profile (even though it should affect only "target_os=chromeos" builds).
,
Dec 23 2016
That does not happen in production, it only applies when running chromeos build in dev mode on your workstation with set of flags that would be incorrect in real life.
,
Apr 7 2017
The problem is that the session extensions (like "Web Store" or "Bookmark Manager") started being loaded into the initial profile.
,
Apr 11 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/095cadb4ac6310a09f9f76b81fc0ae8784323a87 commit 095cadb4ac6310a09f9f76b81fc0ae8784323a87 Author: emaxx <emaxx@chromium.org> Date: Tue Apr 11 17:07:09 2017 Prevent loading of session extensions into the sign-in profile The session extensions (like "Web Store" or "Bookmark Manager") shouldn't be loaded into the original profile. However, they are loaded currently as long as the --login-manager flag is not specified. This happens in the following cases: * On Chrome OS inside the user session after the browser has crashed at least once; * In tests for Chrome OS that use stub or test user directly, without going through the login screen; * In dev builds for Chrome OS running under Linux, when --login-manager is not passed via command line. The effect of this bug is that: 1. Unwanted extensions were loaded, resulting in extra resources usage; 2. Log spam from signin_screen_policy_provider.cc with messages like "Denying load of Extension ..."; 3. Error popups in tests that try to load extensions via command line when --enable-login-screen-apps was additionally specified. This CL fixes this by extending the condition under which the session extensions are skipped. Now it's also checking whether the profile is the sign-in profile or the initial profile. BUG=576464, 674316 TEST=extended browser tests for file manager extensions Review-Url: https://codereview.chromium.org/2807683003 Cr-Commit-Position: refs/heads/master@{#463661} [modify] https://crrev.com/095cadb4ac6310a09f9f76b81fc0ae8784323a87/chrome/browser/chromeos/file_manager/file_manager_browsertest_base.cc [modify] https://crrev.com/095cadb4ac6310a09f9f76b81fc0ae8784323a87/chrome/browser/extensions/extension_system_impl.cc
,
Apr 13 2017
,
Aug 1 2017
,
Jan 22 2018
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by emmanuel...@gmail.com
, Dec 14 2016