Issue 949143003: Merge to M41 branch 2272: Disable delaying of Apps installation on Chrome OS for first time sign-in

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 949143003: Merge to M41 branch 2272: Disable delaying of Apps installation on Chrome OS for first time sign-in (Closed)

Created:
5 years, 10 months ago by binjin

Modified:
5 years, 10 months ago

Reviewers:
Dmitry Polukhin, not at google - send to devlin

CC:
chromium-reviews, chromium-apps-reviews_chromium.org, extensions-reviews_chromium.org

Base URL:
https://chromium.googlesource.com/chromium/src.git@2272

Target Ref:
refs/pending/branch-heads/2272

Project:
chromium

Visibility:
Public.

More Reviews

Description

Merge to M41 branch 2272: Disable delaying of Apps installation on Chrome OS for first time sign-in This behavior is introduced by a772ed62ff0641df984feffe977b1545e94d53f1. It makes ExternalPrefLoader not being ready when user sign in to Chromebook for the first time and accidentally blocks all installation of external extensions if Chrome Sync is turned off, including force installed extensions by policy. BUG=457539 TEST=manual Review URL: https://codereview.chromium.org/931663007 Cr-Commit-Position: refs/heads/master@{#317074} (cherry picked from commit 9a5194c07bd95febb99e61cdaaf9e42e602bd0be) BUG=457539 TBR=kalman@chromium.org,dpolukhin@chromium.org Committed: https://chromium.googlesource.com/chromium/src/+/405b05280ec938eebb2000874e68dd56a0527f10

Patch Set 1 #

Created: 5 years, 10 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+1 line, -4 lines)			Patch
	M	chrome/browser/extensions/external_provider_impl.cc	View		1 chunk	+1 line, -4 lines	0 comments	Download

Messages

Total messages: 1 (0 generated)

Expand Messages | Collapse Messages

binjin

5 years, 10 months ago (2015-02-24 14:27:29 UTC) #1

Message was sent while issue was closed.

Committed patchset #1 (id:1) manually as
405b05280ec938eebb2000874e68dd56a0527f10 (presubmit successful).

Expand Messages | Collapse Messages