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

Unified Diff: base/synchronization/shared_memory_seqlock_buffer.h

Issue 2358123005: Move OneWriterSeqLock and SharedMemorySeqLockBuffer from content/ to device/base/synchronization (Closed)
Patch Set: Created 4 years, 3 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: base/synchronization/shared_memory_seqlock_buffer.h
diff --git a/content/common/shared_memory_seqlock_buffer.h b/base/synchronization/shared_memory_seqlock_buffer.h
similarity index 71%
rename from content/common/shared_memory_seqlock_buffer.h
rename to base/synchronization/shared_memory_seqlock_buffer.h
index cee4f5cb702ff5f27aa7de7946ce3bcc93656b09..afa3dc8d6fcc0aa9b4f8ad66a920ebf2e4031714 100644
--- a/content/common/shared_memory_seqlock_buffer.h
+++ b/base/synchronization/shared_memory_seqlock_buffer.h
@@ -2,12 +2,12 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CONTENT_COMMON_SHARED_MEMORY_SEQLOCK_BUFFER_H_
-#define CONTENT_COMMON_SHARED_MEMORY_SEQLOCK_BUFFER_H_
+#ifndef BASE_SYNCHRONIZATION_SHARED_MEMORY_SEQLOCK_BUFFER_H_
+#define BASE_SYNCHRONIZATION_SHARED_MEMORY_SEQLOCK_BUFFER_H_
-#include "content/common/one_writer_seqlock.h"
+#include "base/synchronization/one_writer_seqlock.h"
-namespace content {
+namespace base {
// This structure is stored in shared memory that's shared between the browser
// which does the hardware polling, and the consumers of the data,
@@ -18,13 +18,13 @@ namespace content {
// Writer and reader operate on the same buffer assuming contention is low, and
// contention is detected by using the associated SeqLock.
-template<class Data>
+template <class Data>
class SharedMemorySeqLockBuffer {
timvolodine 2016/09/23 17:34:27 should this actually be in base/? i.e. is it requi
public:
OneWriterSeqLock seqlock;
Data data;
};
-} // namespace content
+} // namespace base
-#endif // CONTENT_COMMON_SHARED_MEMORY_SEQLOCK_BUFFER_H_
+#endif // BASE_SYNCHRONIZATION_SHARED_MEMORY_SEQLOCK_BUFFER_H_

Powered by Google App Engine
This is Rietveld 408576698