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

Side by Side Diff: base/test/test_reg_util_win.cc

Issue 93903011: Remove 'using base::string16;' from base/strings/string16.h and remove temporary (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: asdf Created 6 years, 11 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « base/test/test_reg_util_win.h ('k') | base/test/test_reg_util_win_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/test/test_reg_util_win.h" 5 #include "base/test/test_reg_util_win.h"
6 6
7 #include "base/guid.h" 7 #include "base/guid.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/strings/string_number_conversions.h" 9 #include "base/strings/string_number_conversions.h"
10 #include "base/strings/string_util.h" 10 #include "base/strings/string_util.h"
11 #include "base/strings/utf_string_conversions.h" 11 #include "base/strings/utf_string_conversions.h"
12 #include "testing/gtest/include/gtest/gtest.h" 12 #include "testing/gtest/include/gtest/gtest.h"
13 13
14 namespace registry_util { 14 namespace registry_util {
15 15
16 namespace { 16 namespace {
17 17
18 const wchar_t kTimestampDelimiter[] = L"$"; 18 const wchar_t kTimestampDelimiter[] = L"$";
19 const wchar_t kTempTestKeyPath[] = L"Software\\Chromium\\TempTestKeys"; 19 const wchar_t kTempTestKeyPath[] = L"Software\\Chromium\\TempTestKeys";
20 20
21 void DeleteStaleTestKeys(const base::Time& now, const string16& test_key_root) { 21 void DeleteStaleTestKeys(const base::Time& now,
22 const base::string16& test_key_root) {
22 base::win::RegKey test_root_key; 23 base::win::RegKey test_root_key;
23 if (test_root_key.Open(HKEY_CURRENT_USER, 24 if (test_root_key.Open(HKEY_CURRENT_USER,
24 test_key_root.c_str(), 25 test_key_root.c_str(),
25 KEY_ALL_ACCESS) != ERROR_SUCCESS) { 26 KEY_ALL_ACCESS) != ERROR_SUCCESS) {
26 // This will occur on first-run, but is harmless. 27 // This will occur on first-run, but is harmless.
27 return; 28 return;
28 } 29 }
29 30
30 base::win::RegistryKeyIterator iterator_test_root_key(HKEY_CURRENT_USER, 31 base::win::RegistryKeyIterator iterator_test_root_key(HKEY_CURRENT_USER,
31 test_key_root.c_str()); 32 test_key_root.c_str());
32 for (; iterator_test_root_key.Valid(); ++iterator_test_root_key) { 33 for (; iterator_test_root_key.Valid(); ++iterator_test_root_key) {
33 string16 key_name = iterator_test_root_key.Name(); 34 base::string16 key_name = iterator_test_root_key.Name();
34 std::vector<string16> tokens; 35 std::vector<base::string16> tokens;
35 Tokenize(key_name, string16(kTimestampDelimiter), &tokens); 36 Tokenize(key_name, base::string16(kTimestampDelimiter), &tokens);
36 int64 key_name_as_number = 0; 37 int64 key_name_as_number = 0;
37 38
38 if (!base::StringToInt64(tokens[0], &key_name_as_number)) { 39 if (!base::StringToInt64(tokens[0], &key_name_as_number)) {
39 test_root_key.DeleteKey(key_name.c_str()); 40 test_root_key.DeleteKey(key_name.c_str());
40 continue; 41 continue;
41 } 42 }
42 43
43 base::Time key_time = base::Time::FromInternalValue(key_name_as_number); 44 base::Time key_time = base::Time::FromInternalValue(key_name_as_number);
44 base::TimeDelta age = now - key_time; 45 base::TimeDelta age = now - key_time;
45 46
46 if (age > base::TimeDelta::FromHours(24)) 47 if (age > base::TimeDelta::FromHours(24))
47 test_root_key.DeleteKey(key_name.c_str()); 48 test_root_key.DeleteKey(key_name.c_str());
48 } 49 }
49 } 50 }
50 51
51 string16 GenerateTempKeyPath(const string16& test_key_root, 52 base::string16 GenerateTempKeyPath(const base::string16& test_key_root,
52 const base::Time& timestamp) { 53 const base::Time& timestamp) {
53 string16 key_path = test_key_root; 54 base::string16 key_path = test_key_root;
54 key_path += L"\\" + base::Int64ToString16(timestamp.ToInternalValue()); 55 key_path += L"\\" + base::Int64ToString16(timestamp.ToInternalValue());
55 key_path += kTimestampDelimiter + base::ASCIIToWide(base::GenerateGUID()); 56 key_path += kTimestampDelimiter + base::ASCIIToWide(base::GenerateGUID());
56 57
57 return key_path; 58 return key_path;
58 } 59 }
59 60
60 } // namespace 61 } // namespace
61 62
62 RegistryOverrideManager::ScopedRegistryKeyOverride::ScopedRegistryKeyOverride( 63 RegistryOverrideManager::ScopedRegistryKeyOverride::ScopedRegistryKeyOverride(
63 HKEY override, 64 HKEY override,
64 const string16& key_path) 65 const base::string16& key_path)
65 : override_(override) { 66 : override_(override) {
66 EXPECT_EQ( 67 EXPECT_EQ(
67 ERROR_SUCCESS, 68 ERROR_SUCCESS,
68 temp_key_.Create(HKEY_CURRENT_USER, key_path.c_str(), KEY_ALL_ACCESS)); 69 temp_key_.Create(HKEY_CURRENT_USER, key_path.c_str(), KEY_ALL_ACCESS));
69 EXPECT_EQ(ERROR_SUCCESS, 70 EXPECT_EQ(ERROR_SUCCESS,
70 ::RegOverridePredefKey(override_, temp_key_.Handle())); 71 ::RegOverridePredefKey(override_, temp_key_.Handle()));
71 } 72 }
72 73
73 RegistryOverrideManager:: 74 RegistryOverrideManager::
74 ScopedRegistryKeyOverride::~ScopedRegistryKeyOverride() { 75 ScopedRegistryKeyOverride::~ScopedRegistryKeyOverride() {
75 ::RegOverridePredefKey(override_, NULL); 76 ::RegOverridePredefKey(override_, NULL);
76 temp_key_.DeleteKey(L""); 77 temp_key_.DeleteKey(L"");
77 } 78 }
78 79
79 RegistryOverrideManager::RegistryOverrideManager() 80 RegistryOverrideManager::RegistryOverrideManager()
80 : timestamp_(base::Time::Now()), test_key_root_(kTempTestKeyPath) { 81 : timestamp_(base::Time::Now()), test_key_root_(kTempTestKeyPath) {
81 DeleteStaleTestKeys(timestamp_, test_key_root_); 82 DeleteStaleTestKeys(timestamp_, test_key_root_);
82 } 83 }
83 84
84 RegistryOverrideManager::RegistryOverrideManager(const base::Time& timestamp, 85 RegistryOverrideManager::RegistryOverrideManager(
85 const string16& test_key_root) 86 const base::Time& timestamp,
87 const base::string16& test_key_root)
86 : timestamp_(timestamp), test_key_root_(test_key_root) { 88 : timestamp_(timestamp), test_key_root_(test_key_root) {
87 DeleteStaleTestKeys(timestamp_, test_key_root_); 89 DeleteStaleTestKeys(timestamp_, test_key_root_);
88 } 90 }
89 91
90 RegistryOverrideManager::~RegistryOverrideManager() {} 92 RegistryOverrideManager::~RegistryOverrideManager() {}
91 93
92 void RegistryOverrideManager::OverrideRegistry( 94 void RegistryOverrideManager::OverrideRegistry(
93 HKEY override, 95 HKEY override,
94 const string16& /*override_name*/) { 96 const base::string16& /*override_name*/) {
95 string16 key_path = GenerateTempKeyPath(test_key_root_, timestamp_); 97 base::string16 key_path = GenerateTempKeyPath(test_key_root_, timestamp_);
96 overrides_.push_back(new ScopedRegistryKeyOverride(override, key_path)); 98 overrides_.push_back(new ScopedRegistryKeyOverride(override, key_path));
97 } 99 }
98 100
99 string16 GenerateTempKeyPath() { 101 base::string16 GenerateTempKeyPath() {
100 return GenerateTempKeyPath(string16(kTempTestKeyPath), base::Time::Now()); 102 return GenerateTempKeyPath(base::string16(kTempTestKeyPath),
103 base::Time::Now());
101 } 104 }
102 105
103 } // namespace registry_util 106 } // namespace registry_util
OLDNEW
« no previous file with comments | « base/test/test_reg_util_win.h ('k') | base/test/test_reg_util_win_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698