Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(649)

Issue 2150483004: Plumbing for login apps device policy to extensions. (Closed)

Created:
4 years, 5 months ago by achuithb
Modified:
3 years, 10 months ago
CC:
chromium-reviews, chromium-apps-reviews_chromium.org, extensions-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Plumbing for login apps device policy to extensions. ChromeProcessManagerDelegate: * Enable background pages for apps in the login profile when command-line switch --enable-login-apps is set. ExtensionManagement: * Factor out common code into GetInstallListByMode from GetForceInstallList and GetRecommendedInstallList. * Common function UpdateForcedExtensions. ExtensionSystemImpl: * Enable extensions in login profile when command-line switch --enable-login-apps is set. ExternalProviderImpl: * In CreateExternalProviders, create an ExternalPolicyLoader for login apps in the login profile. ExtensionInstallListPolicyHandler: * Common base class to parse login and force extension install lists. * Switch --enable-login-apps to enable login apps. * Extension pref extensions.install.loginlist. BUG=576464

Patch Set 1 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+182 lines, -63 lines) Patch
M chrome/browser/extensions/chrome_process_manager_delegate.cc View 3 chunks +21 lines, -2 lines 0 comments Download
M chrome/browser/extensions/extension_management.h View 1 chunk +8 lines, -0 lines 0 comments Download
M chrome/browser/extensions/extension_management.cc View 5 chunks +43 lines, -37 lines 0 comments Download
M chrome/browser/extensions/extension_service.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/extensions/extension_system_impl.cc View 2 chunks +6 lines, -0 lines 0 comments Download
M chrome/browser/extensions/external_provider_impl.cc View 1 chunk +14 lines, -0 lines 0 comments Download
M chrome/browser/extensions/policy_handlers.h View 1 chunk +31 lines, -4 lines 0 comments Download
M chrome/browser/extensions/policy_handlers.cc View 3 chunks +39 lines, -20 lines 0 comments Download
M chrome/browser/policy/configuration_policy_handler_list_factory.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M chrome/common/chrome_switches.h View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/common/chrome_switches.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M extensions/browser/extension_prefs.cc View 1 chunk +1 line, -0 lines 0 comments Download
M extensions/browser/pref_names.h View 1 chunk +7 lines, -0 lines 0 comments Download
M extensions/browser/pref_names.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 18 (15 generated)
Denis Kuznetsov (DE-MUC)
This CL is succeeded by cl/2306143002
4 years, 3 months ago (2016-09-02 17:44:26 UTC) #16
achuithb
On 2016/09/02 17:44:26, Denis Kuznetsov (DE-MUC) wrote: > This CL is succeeded by cl/2306143002 https://codereview.chromium.org/2306143002
4 years, 3 months ago (2016-09-02 23:13:48 UTC) #17
achuithb
4 years, 3 months ago (2016-09-02 23:13:51 UTC) #18
On 2016/09/02 17:44:26, Denis Kuznetsov (DE-MUC) wrote:
> This CL is succeeded by cl/2306143002

https://codereview.chromium.org/2306143002

Powered by Google App Engine
This is Rietveld 408576698