Make nt_registry Create/OpenRegKey return a scoped object

chrome/install_static/install_util.cc

 // Copyright 2014 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/install_static/install_util.h"
 
 #include <windows.h>
 #include <assert.h>
 #include <stdlib.h>
 #include <string.h>
@@ skipping 332 matching lines @@
     // If reading the value failed, treat it as though sampling isn't in effect,
     // implicitly meaning this install is in the sample.
     return true;
   }
   return out_value == 1;
 }
 
 bool SetCollectStatsInSample(bool in_sample) {
   std::wstring registry_path = GetChromeInstallRegistryPath();
 
-  HANDLE key_handle = INVALID_HANDLE_VALUE;
-  if (!nt::CreateRegKey(nt::HKCU, registry_path.c_str(),
-                        KEY_SET_VALUE | KEY_WOW64_32KEY, &key_handle)) {
+  nt::ScopedHANDLE key = nt::CreateRegKey(nt::HKCU, registry_path.c_str(),
+                                          KEY_SET_VALUE | KEY_WOW64_32KEY);
+  if (!key.is_valid())

[grt (UTC plus 2), 2017/02/08 21:44:02]

wdyt of an explicit operator bool so this can be written as:
  if (!key)

     return false;
-  }
 
-  bool success = nt::SetRegValueDWORD(key_handle, kRegValueChromeStatsSample,
-                                      in_sample ? 1 : 0);
-  nt::CloseRegKey(key_handle);
-  return success;
+  return nt::SetRegValueDWORD(key.get(), kRegValueChromeStatsSample,

[grt (UTC plus 2), 2017/02/08 21:44:02]

i think that
  return key.SetRegValueDWORD(...);
is quite a bit nicer. is there a reason not to go this route?

+                              in_sample ? 1 : 0);
 }
 
 // Appends "[kCompanyPathName\]kProductPathName[install_suffix]" to |path|,
 // returning a reference to |path|.
 std::wstring& AppendChromeInstallSubDirectory(const InstallConstants& mode,
                                               bool include_suffix,
                                               std::wstring* path) {
   if (*kCompanyPathName) {
     path->append(kCompanyPathName);
     path->push_back(L'\\');
@@ skipping 375 matching lines @@
     case ChannelStrategy::ADDITIONAL_PARAMETERS:
       return ChannelFromAdditionalParameters(mode, system_level);
     case ChannelStrategy::FIXED:
       return mode.default_channel_name;
   }
 
   return std::wstring();
 }
 
 }  // namespace install_static