[media] Do not configure mipmaps in gles2_decoder_helper.

This fails on Mac when run in a headless content_shell.

Also fixes an incorrect thread post bug in VdaVideoDecoder.

Bug: 522298
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I09d397ee108920fc35e8aadc45d9b47840eb8686
Reviewed-on: https://chromium-review.googlesource.com/1033834Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Frank Liberato <liberato@chromium.org>
Commit-Queue: Dan Sanders <sandersd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554555}

media/gpu/command_buffer_helper.h

@@ -53,8 +53,8 @@ class MEDIA_GPU_EXPORT CommandBufferHelper
   // See glTexImage2D() for argument definitions.
   //
   // The texture will be configured as a video frame: linear filtering, clamp to
-  // edge, no mipmaps. If |target| is GL_TEXTURE_2D, storage will be allocated
-  // but not initialized.
+  // edge. If |target| is GL_TEXTURE_2D, storage will be allocated but not
+  // initialized.
   //
   // The context must be current.
   //

media/gpu/gles2_decoder_helper.cc

@@ -89,12 +89,6 @@ class GLES2DecoderHelperImpl : public GLES2DecoderHelper {
                                     texture_ref.get(), GL_TEXTURE_WRAP_T,
                                     GL_CLAMP_TO_EDGE);
 
-    texture_manager_->SetParameteri(__func__, decoder_->GetErrorState(),
-                                    texture_ref.get(), GL_TEXTURE_BASE_LEVEL,
-                                    0);
-    texture_manager_->SetParameteri(__func__, decoder_->GetErrorState(),
-                                    texture_ref.get(), GL_TEXTURE_MAX_LEVEL, 0);
-
     // TODO(sandersd): Do we always want to allocate for GL_TEXTURE_2D?
     if (target == GL_TEXTURE_2D) {
       glTexImage2D(target,           // target

media/gpu/gles2_decoder_helper.h

@@ -42,7 +42,7 @@ class MEDIA_GPU_EXPORT GLES2DecoderHelper {
   virtual bool MakeContextCurrent() = 0;
 
   // Creates a texture and configures it as a video frame (linear filtering,
-  // clamp to edge, no mipmaps). The context must be current.
+  // clamp to edge). The context must be current.
   //
   // See glTexImage2D() for parameter definitions.
   //

media/gpu/ipc/service/vda_video_decoder.cc

@@ -253,9 +253,9 @@ void VdaVideoDecoder::Initialize(
   config_ = config;
 
   if (reinitializing) {
-    gpu_task_runner_->PostTask(FROM_HERE,
-                               base::BindOnce(&VdaVideoDecoder::InitializeDone,
-                                              parent_weak_this_, true));
+    parent_task_runner_->PostTask(
+        FROM_HERE, base::BindOnce(&VdaVideoDecoder::InitializeDone,
+                                  parent_weak_this_, true));
     return;
   }