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

Unified Diff: components/user_prefs/tracked/registry_hash_store_contents_win_unittest.cc

Issue 2204943002: Integrate registry_hash_store_contents with the rest of tracked prefs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove a lost include statement Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/user_prefs/tracked/registry_hash_store_contents_win_unittest.cc
diff --git a/components/user_prefs/tracked/registry_hash_store_contents_win_unittest.cc b/components/user_prefs/tracked/registry_hash_store_contents_win_unittest.cc
index 411c0b19e4ac08fd6f5d10aab352e6b777dd46d6..fc9ee8ed62840073234166b5c64f086b754cb78f 100644
--- a/components/user_prefs/tracked/registry_hash_store_contents_win_unittest.cc
+++ b/components/user_prefs/tracked/registry_hash_store_contents_win_unittest.cc
@@ -16,9 +16,12 @@ namespace {
constexpr base::char16 kProfileName[] = L"test_profile";
constexpr base::char16 kStoreKeyName[] = L"Foo\\TestStore";
-// MACs are 32 characters long.
-constexpr char kTestStringA[] = "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa";
-constexpr char kTestStringB[] = "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb";
+// MACs are 32 characters long, which stored in the registry are 64 characters
+// long due to 8-bit to 16-bit char conversion.
gab 2016/08/08 04:37:46 Hmmmm? UTF8ToUTF16 shouldn't change the size of th
proberge 2016/08/31 17:30:17 I asked @grt why he used REG_BINARY instead of REG
+constexpr char kTestStringA[] =
+ "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa";
+constexpr char kTestStringB[] =
+ "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb";
constexpr char kAtomicPrefPath[] = "path1";
constexpr char kSplitPrefPath[] = "extension";
@@ -115,4 +118,4 @@ TEST_F(RegistryHashStoreContentsWinTest, TestReset) {
split_macs.clear();
EXPECT_FALSE(contents->GetSplitMacs(kSplitPrefPath, &split_macs));
EXPECT_EQ(0U, split_macs.size());
-}
+}

Powered by Google App Engine
This is Rietveld 408576698