shell_util: Minor tweaks to comments that didn't land in previous CL.

chrome/installer/util/shell_util.cc

 // Copyright (c) 2012 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.
 //
 // This file defines functions that integrate Chrome in Windows shell. These
 // functions can be used by Chrome as well as Chrome installer. All of the
 // work is done by the local functions defined in anonymous namespace in
 // this class.
 
 #include "chrome/installer/util/shell_util.h"
@@ skipping 138 matching lines @@
   if (suffix_.empty()) {
     NOTREACHED();
     return false;
   }
   suffix->assign(suffix_);
   return true;
 }
 
 // This class represents a single registry entry (a key and its value). A
 // collection of registry entries should be collected into a list and written
-// atomically using a WorkItemList. This is preferred to writing to the registry
-// directly, because if anything goes wrong, they can be rolled back atomically.
+// transactionally using a WorkItemList. This is preferred to writing to the
+// registry directly, because if anything goes wrong, they can be rolled back.
 class RegistryEntry {
  public:
   // A bit-field enum of places to look for this key in the Windows registry.
   enum LookForIn {
     LOOK_IN_HKCU = 1 << 0,
     LOOK_IN_HKLM = 1 << 1,
     LOOK_IN_HKCU_THEN_HKLM = LOOK_IN_HKCU | LOOK_IN_HKLM,
   };
 
   // Identifies the type of removal this RegistryEntry is flagged for, if any.
@@ skipping 34 matching lines @@
     base::string16 application_name;
     base::FilePath application_icon_path;
     int application_icon_index;
     base::string16 application_description;
     base::string16 publisher_name;
 
     // The CLSID for the application's DelegateExecute handler. May be empty.
     base::string16 delegate_clsid;
   };
 
-  // Create a object that represent default value of a key
+  // Create an object that represent default value of a key.
   RegistryEntry(const base::string16& key_path, const base::string16& value);
 
-  // Create a object that represent a key of type REG_SZ
+  // Create an object that represent a key of type REG_SZ.
   RegistryEntry(const base::string16& key_path,
                 const base::string16& name,
                 const base::string16& value);
 
-  // Create a object that represent a key of integer type
+  // Create an object that represent a key of integer type.
   RegistryEntry(const base::string16& key_path,
                 const base::string16& name,
                 DWORD value);
 
   // Returns the Windows browser client registration key for Chrome.  For
   // example: "Software\Clients\StartMenuInternet\Chromium[.user]".  Strictly
   // speaking, we should use the name of the executable (e.g., "chrome.exe"),
   // but that ship has sailed.  The cost of switching now is re-prompting users
   // to make Chrome their default browser, which isn't polite.  |suffix| is the
   // user-specific registration suffix; see GetUserSpecificDefaultBrowserSuffix
@@ skipping 2365 matching lines @@
   base::string16 key_path(ShellUtil::kRegClasses);
   key_path.push_back(base::FilePath::kSeparators[0]);
   key_path.append(prog_id);
   return InstallUtil::DeleteRegistryKey(
       HKEY_CURRENT_USER, key_path, WorkItem::kWow64Default);
 
   // TODO(mgiuca): Remove the extension association entries. This requires that
   // the extensions associated with a particular prog_id are stored in that
   // prog_id's key.
 }