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

Unified Diff: components/browser_watcher/watcher_win.h

Issue 743463002: Browser watcher, part deux. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkgr
Patch Set: Address Erik's remaining comments. Created 6 years, 1 month 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: components/browser_watcher/watcher_win.h
diff --git a/components/browser_watcher/watcher_win.h b/components/browser_watcher/watcher_win.h
deleted file mode 100644
index bdea55a8598902495d5c43058f18d7255e203435..0000000000000000000000000000000000000000
--- a/components/browser_watcher/watcher_win.h
+++ /dev/null
@@ -1,62 +0,0 @@
-// Copyright (c) 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.
-#ifndef COMPONENTS_BROWSER_WATCHER_WATCHER_WIN_H_
-#define COMPONENTS_BROWSER_WATCHER_WATCHER_WIN_H_
-
-#include "base/macros.h"
-#include "base/process/process_handle.h"
-#include "base/strings/string16.h"
-#include "base/time/time.h"
-#include "base/win/scoped_handle.h"
-
-namespace base {
-class CommandLine;
-}
-
-namespace browser_watcher {
-
-// Watches for the exit code of a process and records it in a given registry
-// location.
-class ExitCodeWatcher {
- public:
- // Name of the switch used for the parent process handle.
- static const char kParenthHandleSwitch[];
-
- // Initialize the watcher with a registry path.
- explicit ExitCodeWatcher(const base::char16* registry_path);
- ~ExitCodeWatcher();
-
- // Initializes from arguments on |cmd_line|, returns true on success.
- // This function expects the process handle indicated by kParentHandleSwitch
- // in |cmd_line| to be open with sufficient privilege to wait and retrieve
- // the process exit code.
- // It checks the handle for validity and takes ownership of it.
- // The intent is for this handle to be inherited into the watcher process
- // hosting the instance of this class.
- bool ParseArguments(const base::CommandLine& cmd_line);
-
- // Waits for the process to exit and records its exit code in registry.
- // This is a blocking call.
- void WaitForExit();
-
- base::ProcessHandle process() const { return process_.Get(); }
-
- private:
- // Writes |exit_code| to registry, returns true on success.
- bool WriteProcessExitCode(int exit_code);
-
- // The registry path the exit codes are written to.
- base::string16 registry_path_;
-
- // Handle, PID, and creation time of the watched process.
- base::win::ScopedHandle process_;
- base::ProcessId process_pid_;
- base::Time process_creation_time_;
-
- DISALLOW_COPY_AND_ASSIGN(ExitCodeWatcher);
-};
-
-} // namespace browser_watcher
-
-#endif // COMPONENTS_BROWSER_WATCHER_WATCHER_WIN_H_
« no previous file with comments | « components/browser_watcher/watcher_client_win_unittest.cc ('k') | components/browser_watcher/watcher_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698