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

Side by Side Diff: content/common/gpu/media/vt_video_decode_accelerator.cc

Issue 795633005: Add VDA supported profile to GPUInfo. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address patch set 9 review comments Created 5 years, 12 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 <algorithm> 5 #include <algorithm>
6 6
7 #include <CoreVideo/CoreVideo.h> 7 #include <CoreVideo/CoreVideo.h>
8 #include <OpenGL/CGLIOSurface.h> 8 #include <OpenGL/CGLIOSurface.h>
9 #include <OpenGL/gl.h> 9 #include <OpenGL/gl.h>
10 10
11 #include "base/bind.h" 11 #include "base/bind.h"
12 #include "base/command_line.h" 12 #include "base/command_line.h"
13 #include "base/cpu.h"
13 #include "base/logging.h" 14 #include "base/logging.h"
14 #include "base/mac/mac_logging.h" 15 #include "base/mac/mac_logging.h"
15 #include "base/metrics/histogram_macros.h" 16 #include "base/metrics/histogram_macros.h"
16 #include "base/sys_byteorder.h" 17 #include "base/sys_byteorder.h"
17 #include "base/thread_task_runner_handle.h" 18 #include "base/thread_task_runner_handle.h"
18 #include "content/common/gpu/media/vt_video_decode_accelerator.h" 19 #include "content/common/gpu/media/vt_video_decode_accelerator.h"
19 #include "content/public/common/content_switches.h" 20 #include "content/public/common/content_switches.h"
20 #include "media/base/limits.h" 21 #include "media/base/limits.h"
21 #include "ui/gl/scoped_binders.h" 22 #include "ui/gl/scoped_binders.h"
22 23
(...skipping 888 matching lines...) Expand 10 before | Expand all | Expand 10 after
911 client_->NotifyEndOfBitstreamBuffer(bitstream_id); 912 client_->NotifyEndOfBitstreamBuffer(bitstream_id);
912 assigned_bitstream_ids_.clear(); 913 assigned_bitstream_ids_.clear();
913 state_ = STATE_DESTROYING; 914 state_ = STATE_DESTROYING;
914 QueueFlush(TASK_DESTROY); 915 QueueFlush(TASK_DESTROY);
915 } 916 }
916 917
917 bool VTVideoDecodeAccelerator::CanDecodeOnIOThread() { 918 bool VTVideoDecodeAccelerator::CanDecodeOnIOThread() {
918 return false; 919 return false;
919 } 920 }
920 921
922 // static
923 media::VideoDecodeAccelerator::SupportedResolution
924 VTVideoDecodeAccelerator::GetSupportedResolution() {
925 media::VideoDecodeAccelerator::SupportedResolution resolution;
926 resolution.min.SetSize(16, 16);
sandersd (OOO until July 31) 2015/01/05 19:12:04 Below 480P, VideoToolbox uses an internal software
henryhsu 2015/01/13 05:40:44 Done.
sandersd (OOO until July 31) 2015/01/13 18:02:15 We have done additional testing, and most hardware
927 base::CPU cpu;
928 // Ivy Bridge+ platforms can support more than 1920x1080.
sandersd (OOO until July 31) 2015/01/05 19:12:04 As far as I know, this has no bearing on the resol
henryhsu 2015/01/13 05:40:44 The code of cpu detection is for chromeos. Could y
sandersd (OOO until July 31) 2015/01/13 18:02:15 I'm not certain, all the hardware we have tested h
929 if ((cpu.vendor_name() == "GenuineIntel") && cpu.model() >= 55)
930 resolution.max.SetSize(4096, 2160);
931 else
932 resolution.max.SetSize(1920, 1088);
933 return resolution;
934 }
935
921 } // namespace content 936 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698