More YVU_420 buffers support.

ui/ozone/platform/drm/gpu/gbm_buffer.cc

 // Copyright 2014 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 "ui/ozone/platform/drm/gpu/gbm_buffer.h"
 
 #include <drm.h>
 #include <fcntl.h>
 #include <gbm.h>
 #include <xf86drm.h>
@@ skipping 146 matching lines @@
 void GbmPixmap::SetProcessingCallback(
     const ProcessingCallback& processing_callback) {
   DCHECK(processing_callback_.is_null());
   processing_callback_ = processing_callback;
 }
 
 gfx::NativePixmapHandle GbmPixmap::ExportHandle() {
   gfx::NativePixmapHandle handle;
   for (size_t i = 0;
        i < gfx::NumberOfPlanesForBufferFormat(buffer_->GetFormat()); ++i) {
-    base::ScopedFD scoped_fd(HANDLE_EINTR(dup(buffer_->GetFd(i))));
-    if (!scoped_fd.is_valid()) {
-      PLOG(ERROR) << "dup";
-      return gfx::NativePixmapHandle();
+    if (i < buffer_->GetFdCount()) {

[dnicoara, 2016/06/13 13:41:17]

nit: Would you mind adding a comment explaining why this is done? It'll be
easier for future readers to understand without going through a chain of
codereviews.

[Daniele Castagna, 2016/06/13 17:07:58]

Done.

+      base::ScopedFD scoped_fd(HANDLE_EINTR(dup(buffer_->GetFd(i))));
+      if (!scoped_fd.is_valid()) {
+        PLOG(ERROR) << "dup";
+        return gfx::NativePixmapHandle();
+      }
+      handle.fds.emplace_back(
+          base::FileDescriptor(scoped_fd.release(), true /* auto_close */));
     }
-    handle.fds.emplace_back(
-        base::FileDescriptor(scoped_fd.release(), true /* auto_close */));
     handle.strides_and_offsets.emplace_back(buffer_->GetStride(i),
                                             buffer_->GetOffset(i));
   }
   return handle;
 }
 
 GbmPixmap::~GbmPixmap() {
 }
 
 void* GbmPixmap::GetEGLClientBuffer() const {
@@ skipping 70 matching lines @@
   DCHECK(!processing_callback_.is_null());
   if (!processing_callback_.Run(this, processed_pixmap_)) {
     LOG(ERROR) << "Failed processing NativePixmap";
     return nullptr;
   }
 
   return processed_pixmap_->buffer();
 }
 
 }  // namespace ui