Integrate registry_hash_store_contents with the rest of tracked prefs.

components/user_prefs/tracked/registry_hash_store_contents_win_unittest.cc

 // Copyright (c) 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/user_prefs/tracked/registry_hash_store_contents_win.h"
 
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/test/test_reg_util_win.h"
 #include "base/values.h"
 #include "base/win/registry.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 constexpr base::char16 kRegistryPath[] = L"Foo\\TestStore";
 constexpr base::char16 kStoreKey[] = L"test_store_key";
 
-// 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/09/23 14:01:18]

This is still unexpected to me, i.e. this is passed to contents->SetMac() which
takes an std::string and as such I would think expects a 32-char string, no?

[proberge, 2016/09/23 15:24:34]

My bad, I completely misunderstood what was going on. SetMac() is called with a
64-char string. pref_hash_calculator generates a 32-char string, but then
base::HexEncode()s it.

base::HexEncode() turns it into a 64-char string:
https://cs.chromium.org/chromium/src/base/strings/string_number_conversions.c...

[gab, 2016/09/23 15:56:58]

Ah ok, now that makes more sense, thanks!

+constexpr char kTestStringA[] =
+    "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa";
+constexpr char kTestStringB[] =
+    "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb";
 
 constexpr char kAtomicPrefPath[] = "path1";
 constexpr char kSplitPrefPath[] = "extension";
 
 class RegistryHashStoreContentsWinTest : public testing::Test {
  protected:
   RegistryHashStoreContentsWinTest() {}
 
   void SetUp() override {
     registry_override_manager_.OverrideRegistry(HKEY_CURRENT_USER);
@@ skipping 75 matching lines @@
 
   contents->Reset();
 
   stored_mac.clear();
   EXPECT_FALSE(contents->GetMac(kAtomicPrefPath, &stored_mac));
   EXPECT_TRUE(stored_mac.empty());
 
   split_macs.clear();
   EXPECT_FALSE(contents->GetSplitMacs(kSplitPrefPath, &split_macs));
   EXPECT_EQ(0U, split_macs.size());
-}
+}