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

Side by Side Diff: chrome/browser/process_singleton_startup_lock.h

Issue 12096114: Extract locking behaviour from ProcessSingleton. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rename test suite. Created 7 years, 8 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CHROME_BROWSER_PROCESS_SINGLETON_STARTUP_LOCK_H_
6 #define CHROME_BROWSER_PROCESS_SINGLETON_STARTUP_LOCK_H_
7
8 #include <set>
9 #include <utility>
10 #include <vector>
11
12 #include "base/basictypes.h"
13 #include "base/command_line.h"
14 #include "base/files/file_path.h"
15 #include "base/threading/non_thread_safe.h"
16 #include "chrome/browser/process_singleton.h"
17
18 // Provides a ProcessSingleton::NotificationCallback that can queue up
19 // command-line invocations during startup and execute them when startup
20 // completes.
21 //
22 // The object starts in a locked state. |Unlock()| must be called
23 // when the process is prepared to handle command-line invocations.
24 //
25 // Once unlocked, notifications are forwarded to a wrapped NotificationCallback.
26 class ProcessSingletonStartupLock : public base::NonThreadSafe {
Nico 2013/04/18 18:12:13 I think part of my confusing was caused by this be
27 public:
28 explicit ProcessSingletonStartupLock(
29 const ProcessSingleton::NotificationCallback& original_callback);
30 ~ProcessSingletonStartupLock();
31
32 // Returns the ProcessSingleton::NotificationCallback.
33 // The callback is only valid during the lifetime of the
34 // ProcessSingletonStartupLock instance.
35 ProcessSingleton::NotificationCallback AsNotificationCallback();
36
37 // Executes previously queued command-line invocations and allows future
38 // invocations to be executed immediately.
39 void Unlock();
40
41 bool locked() { return locked_; }
42
43 private:
44 typedef std::pair<CommandLine::StringVector, base::FilePath>
45 DelayedStartupMessage;
46
47 bool NotificationCallbackImpl(const CommandLine& command_line,
48 const base::FilePath& current_directory);
49
50 bool locked_;
51 std::vector<DelayedStartupMessage> saved_startup_messages_;
52 ProcessSingleton::NotificationCallback original_callback_;
53
54 DISALLOW_COPY_AND_ASSIGN(ProcessSingletonStartupLock);
55 };
56
57 #endif // CHROME_BROWSER_PROCESS_SINGLETON_STARTUP_LOCK_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698