limit ui refresh rate

src/main/java/eu/siacs/conversations/Config.java

@@ -22,6 +22,7 @@ public final class Config {
 	public static final int MAX_NUM_PAGES = 3;
 
 	public static final int PROGRESS_UI_UPDATE_INTERVAL = 750;
+	public static final int REFRESH_UI_INTERVAL = 500;
 
 	public static final boolean NO_PROXY_LOOKUP = false; //useful to debug ibb
 

src/main/java/eu/siacs/conversations/ui/ConferenceDetailsActivity.java

@@ -144,25 +144,18 @@ public class ConferenceDetailsActivity extends XmppActivity implements OnConvers
 
 	@Override
 	public void onConversationUpdate() {
-		runOnUiThread(new Runnable() {
+		refreshUi();
-
-			@Override
-			public void run() {
-				updateView();
-			}
-		});
 	}
 
 	@Override
 	public void onMucRosterUpdate() {
-		runOnUiThread(new Runnable() {
+		refreshUi();
+	}
 
 	@Override
-			public void run() {
+	protected void refreshUiReal() {
 		updateView();
 	}
-		});
-	}
 
 	@Override
 	protected void onCreate(Bundle savedInstanceState) {

src/main/java/eu/siacs/conversations/ui/ContactDetailsActivity.java

@@ -139,27 +139,19 @@ public class ContactDetailsActivity extends XmppActivity implements OnAccountUpd
 
 	@Override
 	public void onRosterUpdate() {
-		runOnUiThread(new Runnable() {
+		refreshUi();
-
-			@Override
-			public void run() {
-				invalidateOptionsMenu();
-				populateView();
-			}
-		});
 	}
 
 	@Override
 	public void onAccountUpdate() {
-		runOnUiThread(new Runnable() {
+		refreshUi();
+	}
 
 	@Override
-			public void run() {
+	protected void refreshUiReal() {
 		invalidateOptionsMenu();
 		populateView();
 	}
-		});
-	}
 
 	@Override
 	protected String getShareableUri() {

src/main/java/eu/siacs/conversations/ui/ConversationActivity.java

@@ -1006,24 +1006,7 @@ public class ConversationActivity extends XmppActivity
 	}
 
 	@Override
-	public void onAccountUpdate() {
+	protected void refreshUiReal() {
-		runOnUiThread(new Runnable() {
-
-			@Override
-			public void run() {
-				updateConversationList();
-				ConversationActivity.this.mConversationFragment.updateMessages();
-				updateActionBarTitle();
-			}
-		});
-	}
-
-	@Override
-	public void onConversationUpdate() {
-		runOnUiThread(new Runnable() {
-
-			@Override
-			public void run() {
 		updateConversationList();
 		if (conversationList.size() == 0) {
 			startActivity(new Intent(getApplicationContext(),
@@ -1033,29 +1016,29 @@ public class ConversationActivity extends XmppActivity
 		ConversationActivity.this.mConversationFragment.updateMessages();
 		updateActionBarTitle();
 	}
-		});
+
+	@Override
+	public void onAccountUpdate() {
+		this.refreshUi();
+	}
+
+	@Override
+	public void onConversationUpdate() {
+		this.refreshUi();
 	}
 
 	@Override
 	public void onRosterUpdate() {
-		runOnUiThread(new Runnable() {
+		this.refreshUi();
-
-			@Override
-			public void run() {
-				updateConversationList();
-				ConversationActivity.this.mConversationFragment.updateMessages();
-				updateActionBarTitle();
-			}
-		});
 	}
 
 	@Override
 	public void OnUpdateBlocklist(Status status) {
+		this.refreshUi();
 		runOnUiThread(new Runnable() {
 			@Override
 			public void run() {
 				invalidateOptionsMenu();
-				ConversationActivity.this.mConversationFragment.updateMessages();
 			}
 		});
 	}

src/main/java/eu/siacs/conversations/ui/ManageAccountActivity.java

@@ -32,19 +32,18 @@ public class ManageAccountActivity extends XmppActivity implements OnAccountUpda
 
 	@Override
 	public void onAccountUpdate() {
+		refreshUi();
+	}
+
+	@Override
+	protected void refreshUiReal() {
 		synchronized (this.accountList) {
 			accountList.clear();
 			accountList.addAll(xmppConnectionService.getAccounts());
 		}
-		runOnUiThread(new Runnable() {
-
-			@Override
-			public void run() {
 		invalidateOptionsMenu();
 		mAccountAdapter.notifyDataSetChanged();
 	}
-		});
-	}
 
 	@Override
 	protected void onCreate(Bundle savedInstanceState) {

src/main/java/eu/siacs/conversations/ui/StartConversationActivity.java

@@ -176,15 +176,7 @@ public class StartConversationActivity extends XmppActivity implements OnRosterU
 
 	@Override
 	public void onRosterUpdate() {
-		runOnUiThread(new Runnable() {
+		this.refreshUi();
-
-			@Override
-			public void run() {
-				if (mSearchEditText != null) {
-					filter(mSearchEditText.getText().toString());
-				}
-			}
-		});
 	}
 
 	@Override
@@ -711,16 +703,15 @@ public class StartConversationActivity extends XmppActivity implements OnRosterU
 
 	@Override
 	public void OnUpdateBlocklist(final Status status) {
-		runOnUiThread(new Runnable() {
+		refreshUi();
+	}
 
 	@Override
-			public void run() {
+	protected void refreshUiReal() {
 		if (mSearchEditText != null) {
 			filter(mSearchEditText.getText().toString());
 		}
 	}
-		});
-	}
 
 	public static class MyListFragment extends ListFragment {
 		private AdapterView.OnItemClickListener mOnItemClickListener;

src/main/java/eu/siacs/conversations/ui/VerifyOTRActivity.java

@@ -436,11 +436,11 @@ public class VerifyOTRActivity extends XmppActivity implements XmppConnectionSer
 	}
 
 	public void onConversationUpdate() {
-		runOnUiThread(new Runnable() {
+		refreshUi();
+	}
+
 	@Override
-			public void run() {
+	protected void refreshUiReal() {
 		updateView();
 	}
-		});
-	}
 }

src/main/java/eu/siacs/conversations/ui/XmppActivity.java

@@ -33,7 +33,9 @@ import android.nfc.NfcEvent;
 import android.os.AsyncTask;
 import android.os.Build;
 import android.os.Bundle;
+import android.os.Handler;
 import android.os.IBinder;
+import android.os.SystemClock;
 import android.preference.PreferenceManager;
 import android.text.InputType;
 import android.util.DisplayMetrics;
@@ -99,6 +101,33 @@ public abstract class XmppActivity extends Activity {
 	protected int mTheme;
 	protected boolean mUsingEnterKey = false;
 
+	private long mLastUiRefresh = 0;
+	private Handler mRefreshUiHandler = new Handler();
+	private Runnable mRefreshUiRunnable = new Runnable() {
+		@Override
+		public void run() {
+			mLastUiRefresh = SystemClock.elapsedRealtime();
+			refreshUiReal();
+		}
+	};
+
+
+	protected void refreshUi() {
+		final long diff = SystemClock.elapsedRealtime() - mLastUiRefresh;
+		if (diff > Config.REFRESH_UI_INTERVAL) {
+			mRefreshUiHandler.removeCallbacks(mRefreshUiRunnable);
+			runOnUiThread(mRefreshUiRunnable);
+		} else {
+			final long next = Config.REFRESH_UI_INTERVAL - diff;
+			mRefreshUiHandler.removeCallbacks(mRefreshUiRunnable);
+			mRefreshUiHandler.postDelayed(mRefreshUiRunnable,next);
+		}
+	}
+
+	protected void refreshUiReal() {
+
+	};
+
 	protected interface OnValueEdited {
 		public void onValueEdited(String value);
 	}