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

Issue 10867019: [sync] Add hooks to allow for testing sync credential caching in the presence of strange singleton … (Closed)

Created:
8 years, 4 months ago by Raghu Simha
Modified:
8 years, 4 months ago
CC:
chromium-reviews, ncarter (slow), akalin, tim (not reviewing)
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

[sync] Add hooks to allow for testing sync credential caching in the presence of strange singleton mode Sync credential caching is currently broken by strange singleton mode, due to the fact that the metro and desktop directories are collapsed into one. This patch adds test hooks to CredentialCacheService, to allow for its testing using two side-by-side instances of Desktop Chrome, one using the default desktop user data directory, and the other using what used to be the default metro user data directory. Note that in order to enable these test hooks, the --enable-sync-credential-caching flag will have to be used. If the flag is not used, there will be no change in behavior, and strange singleton mode will continue to work as designed. TBR=thakis@chromium.org BUG=144280 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=153105

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -4 lines) Patch
M chrome/browser/sync/credential_cache_service_factory_win.h View 2 chunks +7 lines, -3 lines 0 comments Download
M chrome/browser/sync/credential_cache_service_factory_win.cc View 2 chunks +19 lines, -1 line 0 comments Download
M chrome/browser/sync/credential_cache_service_win.cc View 3 chunks +15 lines, -0 lines 0 comments Download
M chrome/common/chrome_paths_win.cc View 2 chunks +12 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
Raghu Simha
Carlos/Tim, please review. Thanks.
8 years, 4 months ago (2012-08-22 22:26:36 UTC) #1
cpu_(ooo_6.6-7.5)
lgtm on chrome_paths_win.cc
8 years, 4 months ago (2012-08-23 02:57:35 UTC) #2
Raghu Simha
Tim, could you take a look when you get a chance?
8 years, 4 months ago (2012-08-23 20:53:00 UTC) #3
Raghu Simha
Tim is sheriff today. Paging Nicolas :)
8 years, 4 months ago (2012-08-23 22:58:03 UTC) #4
Nicolas Zea
lgtm
8 years, 4 months ago (2012-08-23 23:05:44 UTC) #5
Raghu Simha
+thakis for OWNERS approval of chrome_paths_win.cc. Changes already reviewed by cpu.
8 years, 4 months ago (2012-08-23 23:34:36 UTC) #6
Nico
8 years, 4 months ago (2012-08-23 23:48:16 UTC) #7
what lgtcpu lgtm

Powered by Google App Engine
This is Rietveld 408576698