Log warnings on loading apps in app_shell |
||||||||||||
Issue descriptionWithout the chrome://extensions page, there is no way to detect warnings when loading extensions (such as invalid permissions). We can LOG(WARNING) the warnings from ShellExtensionSystem::LoadApp(). However, I'm seeing some bogus warnings here, like invalid manifest keys that are actually valid.
,
Oct 4 2016
Example warning message proposed for https://codereview.chromium.org/2390073002/: [96490:96490:1003/183739:1658739220753:WARNING:shell_extension_system.cc(65)] Warnings loading extension at /usr/local/google/home/redacted/tiny-app: Unrecognized manifest key 'minimum_chrome_version'. Permission 'metricsPrivate' is unknown or URL pattern is malformed.
,
Oct 4 2016
,
Oct 6 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6f456fb7856dc567afa9acb31c51cf81793677f0 commit 6f456fb7856dc567afa9acb31c51cf81793677f0 Author: michaelpg <michaelpg@chromium.org> Date: Thu Oct 06 08:10:23 2016 Log app_shell warnings on loading apps Extension warnings should be logged somewhere, since we don't have a UI for them like the Extensions UI's "developer mode". R=steel@chromium.org BUG= 652006 Review-Url: https://codereview.chromium.org/2390073002 Cr-Commit-Position: refs/heads/master@{#423473} [modify] https://crrev.com/6f456fb7856dc567afa9acb31c51cf81793677f0/extensions/shell/browser/shell_extension_system.cc
,
Oct 7 2016
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6f456fb7856dc567afa9acb31c51cf81793677f0 commit 6f456fb7856dc567afa9acb31c51cf81793677f0 Author: michaelpg <michaelpg@chromium.org> Date: Thu Oct 06 08:10:23 2016 Log app_shell warnings on loading apps Extension warnings should be logged somewhere, since we don't have a UI for them like the Extensions UI's "developer mode". R=steel@chromium.org BUG= 652006 Review-Url: https://codereview.chromium.org/2390073002 Cr-Commit-Position: refs/heads/master@{#423473} [modify] https://crrev.com/6f456fb7856dc567afa9acb31c51cf81793677f0/extensions/shell/browser/shell_extension_system.cc
,
Nov 4 2016
[Automated comment] removing mislabelled merge-merged-2840
,
Jan 21 2017
,
Feb 26 2017
,
Mar 4 2017
,
Apr 17 2017
,
May 30 2017
,
Aug 1 2017
,
Oct 14 2017
|
||||||||||||
►
Sign in to add a comment |
||||||||||||
Comment 1 by bugdroid1@chromium.org
, Oct 4 2016