diff options
| author | timurrrr@chromium.org <timurrrr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-17 20:36:29 +0000 |
|---|---|---|
| committer | timurrrr@chromium.org <timurrrr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-17 20:36:29 +0000 |
| commit | c2d4449362cb38bfe492d7655885c4c5b3944139 (patch) | |
| tree | 5393b6d1c019fc3b3ce8c74eea523a0a77f549d9 | |
| parent | 3f7275f851e79ddd8e285e7267e1d8ac7e360f0d (diff) | |
| download | chromium_src-c2d4449362cb38bfe492d7655885c4c5b3944139.zip chromium_src-c2d4449362cb38bfe492d7655885c4c5b3944139.tar.gz chromium_src-c2d4449362cb38bfe492d7655885c4c5b3944139.tar.bz2 | |
Fixed a few data races on reference counters.
BUG=18488
Review URL: http://codereview.chromium.org/215011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26476 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/extensions/user_script_master.h | 2 | ||||
| -rw-r--r-- | chrome/browser/history/history_marshaling.h | 3 | ||||
| -rw-r--r-- | net/base/host_resolver.h | 2 |
3 files changed, 4 insertions, 3 deletions
diff --git a/chrome/browser/extensions/user_script_master.h b/chrome/browser/extensions/user_script_master.h index af66879..57f4fe2 100644 --- a/chrome/browser/extensions/user_script_master.h +++ b/chrome/browser/extensions/user_script_master.h @@ -22,7 +22,7 @@ class StringPiece; // Manages a segment of shared memory that contains the user scripts the user // has installed. Lives on the UI thread. -class UserScriptMaster : public base::RefCounted<UserScriptMaster>, +class UserScriptMaster : public base::RefCountedThreadSafe<UserScriptMaster>, public DirectoryWatcher::Delegate, public NotificationObserver { public: diff --git a/chrome/browser/history/history_marshaling.h b/chrome/browser/history/history_marshaling.h index b289228..947f966 100644 --- a/chrome/browser/history/history_marshaling.h +++ b/chrome/browser/history/history_marshaling.h @@ -19,7 +19,8 @@ namespace history { // Navigation ----------------------------------------------------------------- // Marshalling structure for AddPage. -class HistoryAddPageArgs : public base::RefCounted<HistoryAddPageArgs> { +class HistoryAddPageArgs + : public base::RefCountedThreadSafe<HistoryAddPageArgs> { public: HistoryAddPageArgs(const GURL& arg_url, base::Time arg_time, diff --git a/net/base/host_resolver.h b/net/base/host_resolver.h index e5c46a3..fe91b4d 100644 --- a/net/base/host_resolver.h +++ b/net/base/host_resolver.h @@ -28,7 +28,7 @@ class LoadLog; // request at a time is to create a SingleRequestHostResolver wrapper around // HostResolver (which will automatically cancel the single request when it // goes out of scope). -class HostResolver : public base::RefCounted<HostResolver> { +class HostResolver : public base::RefCountedThreadSafe<HostResolver> { public: // The parameters for doing a Resolve(). |hostname| and |port| are required, // the rest are optional (and have reasonable defaults). |