Rename class to comply with style guide. Tweak comments for grammar and correctness.

chrome/browser/configuration_policy_provider_win.cc

 // Copyright (c) 2010 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 "chrome/browser/configuration_policy_provider_win.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "base/registry.h"
 #include "base/scoped_ptr.h"
 #include "base/sys_string_conversions.h"
 #include "base/values.h"
 
-const wchar_t WinConfigurationPolicyProvider::kHomepageRegistryValueName[] =
+const wchar_t ConfigurationPolicyProviderWin::kHomepageRegistryValueName[] =
     L"Homepage";
-const wchar_t WinConfigurationPolicyProvider::
+const wchar_t ConfigurationPolicyProviderWin::
     kHomepageIsNewTabPageRegistryValueName[] = L"HomepageIsNewTabPage";
-const wchar_t WinConfigurationPolicyProvider::kCookiesModeRegistryValueName[] =
+const wchar_t ConfigurationPolicyProviderWin::kCookiesModeRegistryValueName[] =
     L"CookiesMode";
 
 #if defined(GOOGLE_CHROME_BUILD)
-const wchar_t WinConfigurationPolicyProvider::kPolicyRegistrySubKey[] =
+const wchar_t ConfigurationPolicyProviderWin::kPolicyRegistrySubKey[] =
     L"SOFTWARE\\Policies\\Google\\Google Chrome";
 #else
-const wchar_t WinConfigurationPolicyProvider::kPolicyRegistrySubKey[] =
+const wchar_t ConfigurationPolicyProviderWin::kPolicyRegistrySubKey[] =
     L"SOFTWARE\\Policies\\Chromium";
 #endif
 
-WinConfigurationPolicyProvider::WinConfigurationPolicyProvider() {
+ConfigurationPolicyProviderWin::ConfigurationPolicyProviderWin() {
 }
 
-bool WinConfigurationPolicyProvider::GetRegistryPolicyString(
+bool ConfigurationPolicyProviderWin::GetRegistryPolicyString(
     const wchar_t* value_name, string16* result) {
   DWORD value_size = 0;
   DWORD key_type = 0;
   scoped_array<uint8> buffer;
   RegKey hkcu_policy_key(HKEY_LOCAL_MACHINE, kPolicyRegistrySubKey);
   if (hkcu_policy_key.ReadValue(value_name, 0, &value_size, &key_type)) {
     if (key_type != REG_SZ)
       return false;
     // According to the Microsoft documentation, the string
     // buffer may not be explicitly 0-terminated. Allocate a
@@ skipping 16 matching lines @@
       hklm_policy_key.ReadValue(value_name, buffer.get(), &value_size);
     } else {
       return false;
     }
   }
 
   result->assign(reinterpret_cast<const wchar_t*>(buffer.get()));
   return true;
 }
 
-bool WinConfigurationPolicyProvider::GetRegistryPolicyBoolean(
+bool ConfigurationPolicyProviderWin::GetRegistryPolicyBoolean(
     const wchar_t* value_name, bool* result) {
   DWORD value;
   RegKey hkcu_policy_key(HKEY_LOCAL_MACHINE, kPolicyRegistrySubKey);
   if (hkcu_policy_key.ReadValueDW(value_name, &value)) {
     *result = value != 0;
     return true;
   }
 
   RegKey hklm_policy_key(HKEY_CURRENT_USER, kPolicyRegistrySubKey);
   if (hklm_policy_key.ReadValueDW(value_name, &value)) {
     *result = value != 0;
     return true;
   }
   return false;
 }
 
-bool WinConfigurationPolicyProvider::GetRegistryPolicyInteger(
+bool ConfigurationPolicyProviderWin::GetRegistryPolicyInteger(
     const wchar_t* value_name, uint32* result) {
   DWORD value;
   RegKey hkcu_policy_key(HKEY_LOCAL_MACHINE, kPolicyRegistrySubKey);
   if (hkcu_policy_key.ReadValueDW(value_name, &value)) {
     *result = value;
     return true;
   }
 
   RegKey hklm_policy_key(HKEY_CURRENT_USER, kPolicyRegistrySubKey);
   if (hklm_policy_key.ReadValueDW(value_name, &value)) {
     *result = value;
     return true;
   }
   return false;
 }
 
-const WinConfigurationPolicyProvider::RegistryPolicyMapEntry
-    WinConfigurationPolicyProvider::registry_to_policy_map_[] = {
+const ConfigurationPolicyProviderWin::RegistryPolicyMapEntry
+    ConfigurationPolicyProviderWin::registry_to_policy_map_[] = {
     { Value::TYPE_STRING,
       ConfigurationPolicyStore::kPolicyHomePage,
       kHomepageRegistryValueName },
     { Value::TYPE_BOOLEAN,
       ConfigurationPolicyStore::kPolicyHomepageIsNewTabPage,
       kHomepageIsNewTabPageRegistryValueName },
     { Value::TYPE_INTEGER,
       ConfigurationPolicyStore::kPolicyCookiesMode,
       kCookiesModeRegistryValueName },
 };
 
-bool WinConfigurationPolicyProvider::Provide(
+bool ConfigurationPolicyProviderWin::Provide(
     ConfigurationPolicyStore* store) {
   const RegistryPolicyMapEntry* current;
   const RegistryPolicyMapEntry* end = registry_to_policy_map_ +
       arraysize(registry_to_policy_map_);
 
   for (current = registry_to_policy_map_; current != end; ++current) {
     std::wstring string_value;
     uint32 int_value;
     bool bool_value;
     switch (current->value_type) {
@@ skipping 21 matching lines @@
         break;
       default:
         NOTREACHED();
         return false;
     }
   }
 
   return true;
 }