From c196b262ee3eb3e399a6a2ad6caf6a3ecd74e686 Mon Sep 17 00:00:00 2001
From: Dulmandakh <dulmandakh@gmail.com>
Date: Sat, 16 Mar 2019 20:34:47 +0800
Subject: [PATCH 1/2] fix switch track color on Android. fixes #23962

---
 RNTester/js/SwitchExample.js                    |  4 ++++
 .../react/views/switchview/ReactSwitch.java     | 17 +++++++++++------
 2 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/RNTester/js/SwitchExample.js b/RNTester/js/SwitchExample.js
index ca36372e3788be..f2affa18ef70d7 100644
--- a/RNTester/js/SwitchExample.js
+++ b/RNTester/js/SwitchExample.js
@@ -54,6 +54,10 @@ class BasicSwitchExample extends React.Component<
           <Switch
             testID="on-off-initial-off"
             onValueChange={value => this.setState({falseSwitchIsOn: value})}
+            trackColor={{
+              true: "yellow",
+              false: "purple",
+            }}
             value={this.state.falseSwitchIsOn}
           />
           <OnOffIndicator
diff --git a/ReactAndroid/src/main/java/com/facebook/react/views/switchview/ReactSwitch.java b/ReactAndroid/src/main/java/com/facebook/react/views/switchview/ReactSwitch.java
index 4dd299ccaa43ee..acaaf1b96feca9 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/views/switchview/ReactSwitch.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/views/switchview/ReactSwitch.java
@@ -36,6 +36,7 @@ public void setChecked(boolean checked) {
     if (mAllowChange && isChecked() != checked) {
       mAllowChange = false;
       super.setChecked(checked);
+      setTrackColor(checked);
     }
   }
 
@@ -59,12 +60,7 @@ public void setThumbColor(@Nullable Integer color) {
     // If the switch has a different value than the value sent by JS, we must change it.
     if (isChecked() != on) {
       super.setChecked(on);
-      if (mTrackColorForTrue != null || mTrackColorForFalse != null) {
-        // Update the track color to reflect the new value. We only want to do this if these
-        // props were actually set from JS; otherwise we'll just reset the color to the default.
-        Integer currentTrackColor = on ? mTrackColorForTrue : mTrackColorForFalse;
-        setTrackColor(currentTrackColor);
-      }
+      setTrackColor(on);
     }
     mAllowChange = true;
   }
@@ -90,4 +86,13 @@ public void setTrackColorForFalse(@Nullable Integer color) {
       setTrackColor(mTrackColorForFalse);
     }
   }
+
+  private void setTrackColor(boolean checked) {
+    if (mTrackColorForTrue != null || mTrackColorForFalse != null) {
+      // Update the track color to reflect the new value. We only want to do this if these
+      // props were actually set from JS; otherwise we'll just reset the color to the default.
+      Integer currentTrackColor = checked ? mTrackColorForTrue : mTrackColorForFalse;
+      setTrackColor(currentTrackColor);
+    }
+  }
 }

From 3dad8500ae238126cda7a19b28f694c5d5d3c5c5 Mon Sep 17 00:00:00 2001
From: Dulmandakh <dulmandakh@gmail.com>
Date: Sat, 16 Mar 2019 20:45:00 +0800
Subject: [PATCH 2/2] use single-quote for strings

---
 RNTester/js/SwitchExample.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/RNTester/js/SwitchExample.js b/RNTester/js/SwitchExample.js
index f2affa18ef70d7..d1d81811b2b7c0 100644
--- a/RNTester/js/SwitchExample.js
+++ b/RNTester/js/SwitchExample.js
@@ -55,8 +55,8 @@ class BasicSwitchExample extends React.Component<
             testID="on-off-initial-off"
             onValueChange={value => this.setState({falseSwitchIsOn: value})}
             trackColor={{
-              true: "yellow",
-              false: "purple",
+              true: 'yellow',
+              false: 'purple',
             }}
             value={this.state.falseSwitchIsOn}
           />