Use RegistryOverrideManager properly in RLZ tests.

rlz/lib/rlz_lib.h

Index: rlz/lib/rlz_lib.h
diff --git a/rlz/lib/rlz_lib.h b/rlz/lib/rlz_lib.h
index 1f8be5ca2d54966ea258d4742abfdc1a05a2464d..6e3f451fc1589d358d6276d7df72ddae1d601d5a 100644
--- a/rlz/lib/rlz_lib.h
+++ b/rlz/lib/rlz_lib.h
@@ -52,6 +52,22 @@ class URLRequestContextGetter;
 
 namespace rlz_lib {
 
+// All functions return true on success and false on error.

[grt (UTC plus 2), 2013/11/13 04:23:47]

rogerta: i extracted this comment from win/lib/rlz_lib.h (which i've deleted).
it seemed to fit here.

[Roger Tawa OOO till Jul 10th, 2013/11/13 18:39:31]

Looks fine.  This file still contains a lot of windows specific comments even
though its used on all platforms.

+// This implemenation is thread safe.
+//
+// Each prototype mentions the registry access requirements:
+//
+// HKLM read:  Will work from any process and at any privilege level on Vista.
+// HKCU read:  Calls made from the SYSTEM account must pass the current user's
+//             SID as the optional 'sid' param. Can be called from low integrity
+//             process on Vista.
+// HKCU write: Calls made from the SYSTEM account must pass the current user's
+//             SID as the optional 'sid' param. Calls require at least medium
+//             integrity on Vista (e.g. Toolbar will need to use their broker)
+// HKLM write: Calls must be made from an account with admin rights. No SID
+//             need be passed when running as SYSTEM.
+// Functions which do not access registry will be marked with "no restrictions".
+
 class ScopedRlzValueStoreLock;
 
 // The maximum length of an access points RLZ in bytes.