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

Unified Diff: chrome_frame/registry_watcher.cc

Issue 7065024: Add a self-destruct mechanism to user-level Chrome Frame when it detects that system-level Chrome... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome_frame/registry_watcher.cc
===================================================================
--- chrome_frame/registry_watcher.cc (revision 0)
+++ chrome_frame/registry_watcher.cc (revision 0)
@@ -0,0 +1,85 @@
+// Copyright (c) 2011 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.
+//
+// A utility class that makes it easy to register for registry change
+// notifications.
+//
+
+#include "chrome_frame/registry_watcher.h"
+
+#include "chrome_frame/chrome_frame_helper_util.h"
+
+namespace {
+const wchar_t kRegistryWatcherEventName[] = L"chrome_registry_watcher_event";
+} // namespace
+
+RegistryWatcher::RegistryWatcher(HKEY hive,
+ const wchar_t* path,
+ WAITORTIMERCALLBACK callback)
+ : callback_(callback),
+ wait_event_(NULL),
+ wait_handle_(NULL),
+ stopping_(false) {
+ // Enforce that we can open the given registry path with the KEY_NOTIFY
+ // permission.
+ LONG result = RegOpenKeyEx(hive, path, 0, KEY_NOTIFY, &registry_key_);
+ if (result != ERROR_SUCCESS) {
+ registry_key_ = NULL;
+ }
+}
+
+RegistryWatcher::~RegistryWatcher() {
+ StopWatching();
+ if (registry_key_) {
+ RegCloseKey(registry_key_);
+ registry_key_ = NULL;
+ }
+}
+
+bool RegistryWatcher::StartWatching() {
+ if (!registry_key_ || wait_event_ || !callback_) {
+ return false;
+ }
+
+ bool result = false;
+ wait_event_ = CreateEvent(NULL,
+ FALSE, // Auto-resets
+ FALSE, // Initially non-signalled
+ kRegistryWatcherEventName);
+ if (wait_event_ != NULL) {
+ LONG notify_result = RegNotifyChangeKeyValue(
+ registry_key_,
+ TRUE, // Watch subtree
+ REG_NOTIFY_CHANGE_NAME, // Notifies if a subkey is added.
+ wait_event_,
+ TRUE); // Asynchronous, signal the event when a change occurs.
+
+ if (notify_result == ERROR_SUCCESS) {
+ if (RegisterWaitForSingleObject(&wait_handle_,
+ wait_event_,
+ callback_,
+ reinterpret_cast<void*>(this),
+ INFINITE,
+ WT_EXECUTEDEFAULT)) {
+ stopping_ = false;
+ result = true;
+ }
grt (UTC plus 2) 2011/05/26 17:08:58 else { CloseHandle(wait_event_); wait_event_ = NUL
robertshield 2011/05/27 03:34:59 Need to do this if either the RegisterWait..() or
+ }
+ }
+
+ return result;
+}
+
+void RegistryWatcher::StopWatching() {
+ stopping_ = true;
+ if (wait_handle_) {
+ // Unregister the wait and block until any current handlers have returned.
+ UnregisterWaitEx(wait_handle_, INVALID_HANDLE_VALUE);
+ wait_handle_ = NULL;
+ }
+ if (wait_event_) {
+ CloseHandle(wait_event_);
+ wait_event_ = NULL;
+ }
+}
Property changes on: chrome_frame\registry_watcher.cc
___________________________________________________________________
Added: svn:eol-style
+ LF

Powered by Google App Engine
This is Rietveld 408576698