commit 92d2cc2f15c12e1aab4c1f517bc09e596d47b52a
author nift4 <nift4@protonmail.com> Fri May 12 23:20:56 2023 +0200
committer nift4 <nift4@protonmail.com> Sun Jul 23 18:01:42 2023 +0200
tree 13500e8de4f8ec8535c96cea8748d56c525cac1f
parent d6259b86acfcd08ca1adacb0176912bd3b93100e

FMRadio: fix record activity timer being off after a while

We consume Binder time every second when we check storage space -
calculate that into the Timer to avoid updating far off. Fixes time
jumping from 0 to 2.

Change-Id: Icc02735b8e32f75273c5eec4e1fe87a48bd95eb6

src/com/android/fmradio/FmRecordActivity.java

diff --git a/src/com/android/fmradio/FmRecordActivity.java b/src/com/android/fmradio/FmRecordActivity.java
index 21b957f..4505861 100644
--- a/src/com/android/fmradio/FmRecordActivity.java
+++ b/src/com/android/fmradio/FmRecordActivity.java
@@ -65,7 +65,7 @@
     private static final int TIME_BASE = 60;
     private static final String CHANNEL_ID = "record";
     private Context mContext;
-    private TextView mMintues;
+    private TextView mMinutes;
     private TextView mSeconds;
     private TextView mFrequency;
     private View mStationInfoLayout;
@@ -89,7 +89,7 @@
         mFragmentManager = getFragmentManager();
         setContentView(R.layout.fm_record_activity);
 
-        mMintues = (TextView) findViewById(R.id.minutes);
+        mMinutes = (TextView) findViewById(R.id.minutes);
         mSeconds = (TextView) findViewById(R.id.seconds);
 
         mFrequency = (TextView) findViewById(R.id.frequency);
@@ -408,24 +408,28 @@
     private final Handler mHandler = new Handler() {
         @Override
         public void handleMessage(Message msg) {
+            long recordTimeInMillis = 0;
             switch (msg.what) {
                 case FmListener.MSGID_REFRESH:
                     if (mService != null) {
-                        long recordTimeInMillis = mService.getRecordTime();
-                        long recordTimeInSec = recordTimeInMillis / 1000L;
-                        mMintues.setText(addPaddingForString(recordTimeInSec / TIME_BASE));
-                        mSeconds.setText(addPaddingForString(recordTimeInSec % TIME_BASE));
                         checkStorageSpaceAndStop();
+                        recordTimeInMillis = mService.getRecordTime();
+                        long recordTimeInSec = recordTimeInMillis / 1000L;
+                        mMinutes.setText(addPaddingForString(recordTimeInSec / TIME_BASE));
+                        mSeconds.setText(addPaddingForString(recordTimeInSec % TIME_BASE));
                     }
-                    mHandler.sendEmptyMessageDelayed(FmListener.MSGID_REFRESH, 1000);
+                    mHandler.sendEmptyMessageDelayed(FmListener.MSGID_REFRESH,
+                            1000 - (recordTimeInMillis % 1000));
                     break;
 
                 case MSG_UPDATE_NOTIFICATION:
                     if (mService != null) {
-                        updateRecordingNotification(mService.getRecordTime());
                         checkStorageSpaceAndStop();
+                        recordTimeInMillis = mService.getRecordTime();
+                        updateRecordingNotification(recordTimeInMillis);
                     }
-                    mHandler.sendEmptyMessageDelayed(MSG_UPDATE_NOTIFICATION, 1000);
+                    mHandler.sendEmptyMessageDelayed(MSG_UPDATE_NOTIFICATION,
+                            1000 - (recordTimeInMillis % 1000));
                     break;
 
                 case FmListener.LISTEN_RECORDSTATE_CHANGED: