Issue 8921011: Merge 113852 - Fix failure to disable cloud print connector.

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 8921011: Merge 113852 - Fix failure to disable cloud print connector. (Closed)

Created:
9 years ago by Albert Bodenhamer

Modified:
9 years ago

Reviewers:
Albert Bodenhamer

CC:
chromium-reviews

Base URL:
svn://svn.chromium.org/chrome/branches/963/src/

Visibility:
Public.

Merge 113852 - Fix failure to disable cloud print connector. Fix issue where a crash in the cloud print connector on shutdown will prevent the enable pref change from being persisted. Moves the write of the pref to earlier in the shutdown process so that if a crash occurs the pref will still be recorded. BUG=106919 TEST=Verify 106919 Review URL: http://codereview.chromium.org/8879040 TBR=abodenha@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114040

Created: 9 years ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+1 line, -1 line)			Patch
	M	chrome/service/cloud_print/cloud_print_proxy.cc	View		1 chunk	+1 line, -1 line	0 comments	Download

Total messages: 2 (0 generated)

lgtm