Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. | 2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. |
| 3 * | 3 * |
| 4 * Use of this source code is governed by a BSD-style license | 4 * Use of this source code is governed by a BSD-style license |
| 5 * that can be found in the LICENSE file in the root of the source | 5 * that can be found in the LICENSE file in the root of the source |
| 6 * tree. An additional intellectual property rights grant can be found | 6 * tree. An additional intellectual property rights grant can be found |
| 7 * in the file PATENTS. All contributing project authors may | 7 * in the file PATENTS. All contributing project authors may |
| 8 * be found in the AUTHORS file in the root of the source tree. | 8 * be found in the AUTHORS file in the root of the source tree. |
| 9 */ | 9 */ |
| 10 | 10 |
| 11 /* | 11 /* |
| 12 * Interface to the LibYuv scaling functionality | 12 * Interface to the LibYuv scaling functionality |
| 13 */ | 13 */ |
| 14 | 14 |
| 15 #ifndef WEBRTC_COMMON_VIDEO_LIBYUV_INCLUDE_SCALER_H_ | 15 #ifndef WEBRTC_COMMON_VIDEO_LIBYUV_INCLUDE_SCALER_H_ |
| 16 #define WEBRTC_COMMON_VIDEO_LIBYUV_INCLUDE_SCALER_H_ | 16 #define WEBRTC_COMMON_VIDEO_LIBYUV_INCLUDE_SCALER_H_ |
| 17 | 17 |
| 18 #include "webrtc/common_video/include/i420_buffer_pool.h" | 18 #include "webrtc/common_video/include/i420_buffer_pool.h" |
| 19 #include "webrtc/common_video/libyuv/include/webrtc_libyuv.h" | 19 #include "webrtc/common_video/libyuv/include/webrtc_libyuv.h" |
| 20 #include "webrtc/typedefs.h" | 20 #include "webrtc/typedefs.h" |
| 21 #include "webrtc/video_frame.h" | 21 #include "webrtc/video_frame.h" |
| 22 | 22 |
| 23 namespace webrtc { | 23 namespace webrtc { |
| 24 | 24 |
| 25 // Supported scaling types | 25 // Supported scaling types |
| 26 // Note: Must have the same values as libyuv::FilterMode. | |
|
ossu
2016/06/03 11:50:32
So, the problem was it picked the wrong scaling mo
nisse-webrtc
2016/06/03 11:54:54
That's my understanding, yes.
ossu
2016/06/03 12:37:15
Ah! Then this one-liner really seems like the prud
| |
| 26 enum ScaleMethod { | 27 enum ScaleMethod { |
| 27 kScalePoint, // no interpolation | 28 kScalePoint, // no interpolation |
| 29 kFilterLinear, | |
| 28 kScaleBilinear, | 30 kScaleBilinear, |
| 29 kScaleBox | 31 kScaleBox |
| 30 }; | 32 }; |
| 31 | 33 |
| 32 class Scaler { | 34 class Scaler { |
| 33 public: | 35 public: |
| 34 Scaler(); | 36 Scaler(); |
| 35 ~Scaler(); | 37 ~Scaler(); |
| 36 | 38 |
| 37 // Set interpolation properties: | 39 // Set interpolation properties: |
| (...skipping 22 matching lines...) Expand all Loading... | |
| 60 int src_height_; | 62 int src_height_; |
| 61 int dst_width_; | 63 int dst_width_; |
| 62 int dst_height_; | 64 int dst_height_; |
| 63 bool set_; | 65 bool set_; |
| 64 I420BufferPool buffer_pool_; | 66 I420BufferPool buffer_pool_; |
| 65 }; | 67 }; |
| 66 | 68 |
| 67 } // namespace webrtc | 69 } // namespace webrtc |
| 68 | 70 |
| 69 #endif // WEBRTC_COMMON_VIDEO_LIBYUV_INCLUDE_SCALER_H_ | 71 #endif // WEBRTC_COMMON_VIDEO_LIBYUV_INCLUDE_SCALER_H_ |
| OLD | NEW |