media/gpu: switch v4l2_jpeg_decode_accelerator to use multi-planar APIs

media/gpu/jpeg_decode_accelerator_unittest.cc

 // Copyright 2015 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.
 
 // This has to be included first.
 // See http://code.google.com/p/googletest/issues/detail?id=371
 #include "testing/gtest/include/gtest/gtest.h"
 
 #include <stddef.h>
 #include <stdint.h>
@@ skipping 119 matching lines @@
                        ClientStateNotification<ClientState>* note)
     : test_image_files_(test_image_files), state_(CS_CREATED), note_(note) {}
 
 JpegClient::~JpegClient() {}
 
 void JpegClient::CreateJpegDecoder() {
 #if defined(OS_CHROMEOS) && defined(ARCH_CPU_X86_FAMILY)
   decoder_.reset(
       new VaapiJpegDecodeAccelerator(base::ThreadTaskRunnerHandle::Get()));
 #elif defined(OS_CHROMEOS) && defined(USE_V4L2_CODEC)
-  scoped_refptr<V4L2Device> device =
-      V4L2Device::Create(V4L2Device::kJpegDecoder);
+  scoped_refptr<V4L2Device> device = V4L2Device::Create();

[wuchengli, 2016/12/15 09:18:25]

Hmm. This means JDA is not run by autotest... I need to file a bug.

   if (!device.get()) {
     LOG(ERROR) << "V4L2Device::Create failed";
     SetState(CS_ERROR);
     return;
   }
   decoder_.reset(new V4L2JpegDecodeAccelerator(
       device, base::ThreadTaskRunnerHandle::Get()));
 #else
 #error The JpegDecodeAccelerator is not supported on this platform.
 #endif
@@ skipping 403 matching lines @@
     if (it->first == "jpeg_filenames") {
       jpeg_filenames = it->second.c_str();
       continue;
     }
     if (it->first == "save_to_file") {
       media::g_save_to_file = true;
       continue;
     }
     if (it->first == "v" || it->first == "vmodule")
       continue;
-    LOG(FATAL) << "Unexpected switch: " << it->first << ":" << it->second;
+    LOG(ERROR) << "Unexpected switch: " << it->first << ":" << it->second;

[wuchengli, 2016/12/15 09:18:25]

What's this for?

[jcliang, 2016/12/15 14:55:47]

LOG(FATAL) generates core dumps and it wasted disk space on my DUT. There's no
reason to generate core dump for this error.

+    return 0;
   }
 #if defined(OS_CHROMEOS) && defined(ARCH_CPU_X86_FAMILY)
   media::VaapiWrapper::PreSandboxInitialization();
 #endif
 
   media::g_env = reinterpret_cast<media::JpegDecodeAcceleratorTestEnvironment*>(
       testing::AddGlobalTestEnvironment(
           new media::JpegDecodeAcceleratorTestEnvironment(jpeg_filenames)));
 
   return RUN_ALL_TESTS();
 }