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

Unified Diff: media/base/video_frame.cc

Issue 2843113002: make base::SharedMemoryHandle a class on POSIX. (Closed)
Patch Set: Fix test error. Created 3 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 side-by-side diff with in-line comments
Download patch
Index: media/base/video_frame.cc
diff --git a/media/base/video_frame.cc b/media/base/video_frame.cc
index 5148d003e73c38f01208519386677846b072074f..500842c51b77b8b1ef9f30c63cbac8a9a2ce0194 100644
--- a/media/base/video_frame.cc
+++ b/media/base/video_frame.cc
@@ -712,13 +712,13 @@ VideoFrame::mailbox_holder(size_t texture_index) const {
base::SharedMemoryHandle VideoFrame::shared_memory_handle() const {
DCHECK_EQ(storage_type_, STORAGE_SHMEM);
- DCHECK(shared_memory_handle_ != base::SharedMemory::NULLHandle());
+ DCHECK(shared_memory_handle_.IsValid());
return shared_memory_handle_;
}
size_t VideoFrame::shared_memory_offset() const {
DCHECK_EQ(storage_type_, STORAGE_SHMEM);
- DCHECK(shared_memory_handle_ != base::SharedMemory::NULLHandle());
+ DCHECK(shared_memory_handle_.IsValid());
return shared_memory_offset_;
}

Powered by Google App Engine
This is Rietveld 408576698