DescriptionRemove DeferBackgroundExtensionCreation field trial and supporting code.
This is mostly just "Revert 221621 This defers starting background
extension page RenderViews..." on trunk after resolving conflicts.
r221621 assumes a browser is always created with
StartupBrowserCreatorImpl. However, this is not true for incognito
browsers, some command line switch options, nor some launch flows via
the App Launcher. This meant that background extensions were sometimes
never created.
The field trial setup also needs to be revised, since
it went out to 50x the users that is normal for a field trial on stable
channel.
Field trial is now disabled server-side in m31-33, but there should be
enough data collected in m31 to make a call on whether to attempt this
again with fixes.
This reverts commit 736f456664be5e47df88bf8f936f40ddee299bd1.
> This defers starting background extension page RenderViews
> until after session restore has completed.
>
> This is mainly to help login/startup times so that Chrome is useful to
> the user earlier. To make sure that this is actually helping, this CL
> includes a Finch experiment that will only enable the deferral on 50%
> of the clients.
>
> It is also expected that deferring these will help with some problems
> we've seen when extensions attempt to do GAIA authentication.
>
> BUG=279427, 259791
> TEST=Ran performance tests on ChromeOS and Linux
>
> Review URL: https://chromiumcodereview.appspot.com/23618014
Some conflicts. Most due to
c/b/e/chrome/browser/extensions/extension_process_manager.* moving to
extensions/browser/process_manager.*, and some minor ones elsewhere.
BUG=320787, 288259, 304453
R=mpcomplete@chromium.org, gspencer@chromium.org
TBR=sky@chromium.org, rlp@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=240061
Patch Set 1 #Patch Set 2 : extension_system.h header still needed for GetDisabledPlatformApp #Messages
Total messages: 7 (0 generated)
|