Fix buffer type for JPEG hardware decoder

media/gpu/generic_v4l2_device.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 "media/gpu/generic_v4l2_device.h"
 
 #include <errno.h>
 #include <fcntl.h>
 #include <libdrm/drm_fourcc.h>
@@ skipping 416 matching lines @@
     case Type::kEncoder:
       device_pattern = kEncoderDevicePattern;
       buf_type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
       break;
     case Type::kImageProcessor:
       device_pattern = kImageProcessorDevicePattern;
       buf_type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE;
       break;
     case Type::kJpegDecoder:
       device_pattern = kJpegDecoderDevicePattern;
-      buf_type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE;
+      // TODO(henryhsu): We have to also support MPLANE

[wuchengli, 2016/10/13 09:21:28]

I don't think we need the TODO. It's nice to switch to MPLANE for consistency to
decoder, encoder, and processor. We don't need to support both MPLANE and PLANE.
Decoder, encoder, and processor do not support both, either.

+      buf_type = V4L2_BUF_TYPE_VIDEO_OUTPUT;
       break;
   }
 
   std::vector<std::string> candidate_paths;
 
   // TODO(posciak): Remove this legacy unnumbered device once
   // all platforms are updated to use numbered devices.
   candidate_paths.push_back(device_pattern);
 
   // We are sandboxed, so we can't query directory contents to check which
@@ skipping 38 matching lines @@
   for (const auto& device : devices) {
     if (std::find(device.second.begin(), device.second.end(), pixfmt) !=
         device.second.end())
       return device.first;
   }
 
   return std::string();
 }
 
 }  //  namespace media