[Mojo Video Capture] Add test coverage for accelerated jpeg decoding

media/gpu/BUILD.gn

 # Copyright 2016 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.
 
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//media/gpu/args.gni")
 import("//media/media_options.gni")
 import("//testing/test.gni")
 
@@ skipping 110 matching lines @@
 
   if (is_mac) {
     # On Mac, content/common reaches into here to do some pre-sandbox
     # initialization.
     visibility += [ "//content/common" ]
   }
 
   defines = [ "MEDIA_GPU_IMPLEMENTATION" ]
 
   sources = [
+    "fake_jpeg_decode_accelerator.cc",
+    "fake_jpeg_decode_accelerator.h",

[sandersd (OOO until July 31), 2017/03/14 23:02:34]

Are we sure we want to always build this? It seems dangerous to have this code
accessible in the GPU process if it's not necessary.

[chfremer, 2017/03/15 17:31:05]

I assumed that this would be okay, considering that we also always build
fake_video_decode_accelerator.cc. What is the concrete danger you are seeing?

[sandersd (OOO until July 31), 2017/03/15 17:38:12]

While I don't think either should be in a release buid, FakeVideoDecoder does
nothing but FakeJpegDecoder actually decodes JPEGs.

I do not know the JPEG stack well enough to know if this is an attack surface
increase vs. the regular accelerated path.

     "fake_video_decode_accelerator.cc",
     "fake_video_decode_accelerator.h",
     "gpu_video_accelerator_util.cc",
     "gpu_video_accelerator_util.h",
     "gpu_video_decode_accelerator_factory.cc",
     "gpu_video_decode_accelerator_factory.h",
     "gpu_video_decode_accelerator_helpers.h",
     "shared_memory_region.cc",
     "shared_memory_region.h",
   ]
 
   public_deps = [
     "//base",
     "//gpu",
     "//media",
     "//ui/gfx/geometry",
   ]
   deps = [
+    "//third_party/libyuv:libyuv",
     "//ui/base",
     "//ui/display/types",
     "//ui/gl",
     "//ui/platform_window",
   ]
   libs = []
   ldflags = []
 
   configs += [ ":gpu_config" ]
 
@@ skipping 369 matching lines @@
     "//base/test:test_support",
     "//gpu:test_support",
     "//media/gpu/ipc/common",
     "//media/gpu/ipc/service",
     "//testing/gmock",
     "//testing/gtest",
     "//ui/gfx:test_support",
     "//ui/gfx/geometry",
   ]
 }