From d9116453211b4e1fd2be71f90e1147b7eaebebcb Mon Sep 17 00:00:00 2001
From: Ching Yin Derek Pang <derekpang@google.com>
Date: Sun, 28 Apr 2019 22:54:16 -0700
Subject: [ECO] Fixes in ECOUtils and ECOSession.

1) Fixed data types in ECOUtils.
2) Account dimension for portrait and landscape mode in ECOSession.

Test: Unit test.
Bug: 124805646
Change-Id: Idfd6877bc39b77968d73ee1e95a2573a1439f435
---
 media/eco/include/eco/ECOUtils.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'media/eco/include')

diff --git a/media/eco/include/eco/ECOUtils.h b/media/eco/include/eco/ECOUtils.h
index f9fa129..e5bbfcc 100644
--- a/media/eco/include/eco/ECOUtils.h
+++ b/media/eco/include/eco/ECOUtils.h
@@ -60,7 +60,7 @@ struct SimpleEncoderConfig {
     int32_t mKeyFrameIntervalFrames;
 
     // Frame rate in frames per second. -1 means unavailable.
-    int32_t mFramerateFps;
+    float mFrameRateFps;
 
     SimpleEncoderConfig()
           : mEncoderName(""),
@@ -69,7 +69,7 @@ struct SimpleEncoderConfig {
             mLevel(-1),
             mTargetBitrate(-1),
             mKeyFrameIntervalFrames(-1),
-            mFramerateFps(-1) {}
+            mFrameRateFps(-1) {}
 
     SimpleEncoderConfig(const std::string& name, int32_t codecType, int32_t profile, int32_t level,
                         int32_t bitrate, int32_t kfi, float framerateFps)
@@ -79,7 +79,7 @@ struct SimpleEncoderConfig {
             mLevel(level),
             mTargetBitrate(bitrate),
             mKeyFrameIntervalFrames(kfi),
-            mFramerateFps(framerateFps) {}
+            mFrameRateFps(framerateFps) {}
 
     // Convert this SimpleEncoderConfig to ECOData with dataType.
     ECOData toEcoData(ECOData::ECODatatype dataType);
-- 
cgit v1.2.3