Remove -Wno-unused-private-field clang warning suppression.

content/browser/renderer_host/media/video_capture_buffer_pool.cc

 // Copyright (c) 2013 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.
 
 #include "content/browser/renderer_host/media/video_capture_buffer_pool.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/stl_util.h"
 #include "content/browser/gpu/browser_gpu_memory_buffer_manager.h"
 #include "content/common/gpu/client/gpu_memory_buffer_impl.h"
 #include "content/public/browser/browser_thread.h"
 
 namespace content {
 
 const int VideoCaptureBufferPool::kInvalidId = -1;
 
 // A simple holder of a memory-backed buffer and accesors to it.
 class SimpleBufferHandle final : public VideoCaptureBufferPool::BufferHandle {
  public:
   SimpleBufferHandle(void* data, size_t size, base::SharedMemoryHandle handle)
-      : data_(data), size_(size), handle_(handle) {}
+      : data_(data),
+#if defined(OS_POSIX)
+        handle_(handle),
+#endif
+        size_(size) {}
   ~SimpleBufferHandle() override {}
 
   size_t size() const override { return size_; }
   void* data() override { return data_; }
   ClientBuffer AsClientBuffer() override { return nullptr; }
 #if defined(OS_POSIX)
   base::FileDescriptor AsPlatformFile() override {
 #if defined(OS_MACOSX)
     return handle_.GetFileDescriptor();
 #else
     return handle_;
 #endif  // defined(OS_MACOSX)
   }
 #endif
 
  private:
   void* const data_;
+#if defined(OS_POSIX)
+  const base::SharedMemoryHandle handle_;

[Nico, 2015/07/09 01:58:16]

nit: It's strange to have data_ and size_ not next to each other (I get that the
ifdef is a bit nicer this way, but meh)

[benwells, 2015/07/09 11:43:02]

Done.

+#endif
   const size_t size_;
-  const base::SharedMemoryHandle handle_;
 };
 
 // A holder of a GpuMemoryBuffer-backed buffer, Map()ed on ctor and Unmap()ed on
 // dtor. Holds a weak reference to its GpuMemoryBuffer.
 // TODO(mcasas) Map()ed on ctor, or on first use?
 class GpuMemoryBufferBufferHandle
     final : public VideoCaptureBufferPool::BufferHandle {
  public:
   GpuMemoryBufferBufferHandle(gfx::GpuMemoryBuffer* gmb, size_t size)
       : gmb_(gmb),
@@ skipping 310 matching lines @@
   return buffer_id;
 }
 
 VideoCaptureBufferPool::Tracker* VideoCaptureBufferPool::GetTracker(
     int buffer_id) {
   TrackerMap::const_iterator it = trackers_.find(buffer_id);
   return (it == trackers_.end()) ? NULL : it->second;
 }
 
 }  // namespace content