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

Issue 11576009: Sync: Remove flag to explicitly enable history delete directive sync. (Closed)

Created:
8 years ago by Patrick Dubroy
Modified:
8 years ago
Reviewers:
akalin
CC:
chromium-reviews, Raghu Simha, haitaol1, akalin, tim (not reviewing)
Visibility:
Public.

Description

Sync: Remove flag to explicitly enable history delete directive sync. History delete directives should be on whenever full history sync is on. Added a flag to explicitly disable delete directives, as with other sync data types. Also move flag for disabling tab sync to be alongside all the other sync flags. BUG=165122 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=173429

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -11 lines) Patch
M chrome/browser/sync/profile_sync_components_factory_impl.cc View 1 chunk +5 lines, -3 lines 0 comments Download
M chrome/common/chrome_switches.h View 2 chunks +1 line, -1 line 0 comments Download
M chrome/common/chrome_switches.cc View 3 chunks +7 lines, -7 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Patrick Dubroy
Fred, please take a look.
8 years ago (2012-12-13 12:41:32 UTC) #1
akalin
8 years ago (2012-12-14 20:17:27 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698