Add ThreadConfined(UI) to CatalystInstanceImpl.destroy

Summary:
This already has an assert that `destroy` should only be called on the UI thread. Add an annotation.

Changelog: [Internal]

Reviewed By: mdvacca

Differential Revision: D17989113

fbshipit-source-id: fd44f321cbcb7f0a18383ca6226cce72e5991eea
This commit is contained in:
Joshua Gross
2019-10-18 15:07:11 -07:00
committed by Facebook Github Bot
parent 84915a2d68
commit 526a010ad7
@@ -7,6 +7,7 @@
package com.facebook.react.bridge;
import static com.facebook.infer.annotation.ThreadConfined.UI;
import static com.facebook.systrace.Systrace.TRACE_TAG_REACT_JAVA_BRIDGE;
import android.content.res.AssetManager;
@@ -15,6 +16,7 @@ import android.util.Log;
import androidx.annotation.Nullable;
import com.facebook.common.logging.FLog;
import com.facebook.infer.annotation.Assertions;
import com.facebook.infer.annotation.ThreadConfined;
import com.facebook.jni.HybridData;
import com.facebook.proguard.annotations.DoNotStrip;
import com.facebook.react.bridge.queue.MessageQueueThread;
@@ -326,6 +328,7 @@ public class CatalystInstanceImpl implements CatalystInstance {
* fully shut down other threads.
*/
@Override
@ThreadConfined(UI)
public void destroy() {
Log.d(ReactConstants.TAG, "CatalystInstanceImpl.destroy() start");
UiThreadUtil.assertOnUiThread();