Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(453)

Side by Side Diff: media/gpu/vaapi_video_decode_accelerator.cc

Issue 2399133002: Revert of Attach color space information to hardware decoded NV12 video frames. (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "media/gpu/vaapi_video_decode_accelerator.h" 5 #include "media/gpu/vaapi_video_decode_accelerator.h"
6 6
7 #include <string.h> 7 #include <string.h>
8 8
9 #include <memory> 9 #include <memory>
10 10
(...skipping 414 matching lines...) Expand 10 before | Expand all | Expand 10 after
425 PLATFORM_FAILURE, ); 425 PLATFORM_FAILURE, );
426 426
427 // Notify the client a picture is ready to be displayed. 427 // Notify the client a picture is ready to be displayed.
428 ++num_frames_at_client_; 428 ++num_frames_at_client_;
429 TRACE_COUNTER1("Video Decoder", "Textures at client", num_frames_at_client_); 429 TRACE_COUNTER1("Video Decoder", "Textures at client", num_frames_at_client_);
430 DVLOG(4) << "Notifying output picture id " << output_id 430 DVLOG(4) << "Notifying output picture id " << output_id
431 << " for input " << input_id << " is ready"; 431 << " for input " << input_id << " is ready";
432 // TODO(posciak): Use visible size from decoder here instead 432 // TODO(posciak): Use visible size from decoder here instead
433 // (crbug.com/402760). Passing (0, 0) results in the client using the 433 // (crbug.com/402760). Passing (0, 0) results in the client using the
434 // visible size extracted from the container instead. 434 // visible size extracted from the container instead.
435 // TODO(hubbe): Use the correct color space. http://crbug.com/647725
436 if (client_) 435 if (client_)
437 client_->PictureReady(Picture(output_id, input_id, gfx::Rect(0, 0), 436 client_->PictureReady(
438 gfx::ColorSpace(), picture->AllowOverlay())); 437 Picture(output_id, input_id, gfx::Rect(0, 0), picture->AllowOverlay()));
439 } 438 }
440 439
441 void VaapiVideoDecodeAccelerator::TryOutputSurface() { 440 void VaapiVideoDecodeAccelerator::TryOutputSurface() {
442 DCHECK(task_runner_->BelongsToCurrentThread()); 441 DCHECK(task_runner_->BelongsToCurrentThread());
443 442
444 // Handle Destroy() arriving while pictures are queued for output. 443 // Handle Destroy() arriving while pictures are queued for output.
445 if (!client_) 444 if (!client_)
446 return; 445 return;
447 446
448 if (pending_output_cbs_.empty() || output_buffers_.empty()) 447 if (pending_output_cbs_.empty() || output_buffers_.empty())
(...skipping 1433 matching lines...) Expand 10 before | Expand all | Expand 10 after
1882 return vaapi_pic->dec_surface(); 1881 return vaapi_pic->dec_surface();
1883 } 1882 }
1884 1883
1885 // static 1884 // static
1886 VideoDecodeAccelerator::SupportedProfiles 1885 VideoDecodeAccelerator::SupportedProfiles
1887 VaapiVideoDecodeAccelerator::GetSupportedProfiles() { 1886 VaapiVideoDecodeAccelerator::GetSupportedProfiles() {
1888 return VaapiWrapper::GetSupportedDecodeProfiles(); 1887 return VaapiWrapper::GetSupportedDecodeProfiles();
1889 } 1888 }
1890 1889
1891 } // namespace media 1890 } // namespace media
OLDNEW
« no previous file with comments | « media/gpu/v4l2_video_decode_accelerator.cc ('k') | media/gpu/vt_video_decode_accelerator_mac.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698