V4L2SVDA/VAAPIVDA: use visible size from decoder and pass to client

media/gpu/vp8_decoder.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.
 
 #include "media/gpu/vp8_decoder.h"
 #include "media/base/limits.h"
 
 namespace media {
 
 VP8Decoder::VP8Accelerator::VP8Accelerator() {}
@@ skipping 76 matching lines @@
       // Need a resume point.
       curr_frame_hdr_.reset();
       return kRanOutOfStreamData;
     }
   }
 
   curr_pic_ = accelerator_->CreateVP8Picture();
   if (!curr_pic_)
     return kRanOutOfSurfaces;
 
+  curr_pic_->visible_rect = gfx::Rect(pic_size_);

[Pawel Osciak, 2017/06/19 06:27:40]

Since VP8Parser does not seem to be aligning to 16x16, I think we should make
visible_size use width/height from the header, while pic_size should be that
visible_size aligned to 16x16...

[johnylin1, 2017/06/19 07:13:28]

We will put to an individual CL as discussed in:
https://codereview.chromium.org/2926593002/diff/160001/media/gpu/vp8_decoder....

   if (!DecodeAndOutputCurrentFrame())
     return kDecodeError;
 
   return kRanOutOfStreamData;
 }
 
 void VP8Decoder::RefreshReferenceFrames() {
   if (curr_frame_hdr_->IsKeyframe()) {
     last_frame_ = curr_pic_;
     golden_frame_ = curr_pic_;
@@ skipping 78 matching lines @@
   return pic_size_;
 }
 
 size_t VP8Decoder::GetRequiredNumOfPictures() const {
   const size_t kVP8NumFramesActive = 4;
   const size_t kPicsInPipeline = limits::kMaxVideoFrames + 2;
   return kVP8NumFramesActive + kPicsInPipeline;
 }
 
 }  // namespace media