Skip to content

Commit 8f0cdb0

Browse files
committed
add function to hide profiling tool
1 parent a8343cd commit 8f0cdb0

File tree

1 file changed

+13
-1
lines changed

1 file changed

+13
-1
lines changed

src/components/TestToolsModal.tsx

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import toggleTestToolsModal from '@userActions/TestTool';
1010
import CONST from '@src/CONST';
1111
import ONYXKEYS from '@src/ONYXKEYS';
1212
import ROUTES from '@src/ROUTES';
13+
import { getBrowser, isChromeIOS } from '@libs/Browser';
1314
import Button from './Button';
1415
import ClientSideLoggingToolMenu from './ClientSideLoggingToolMenu';
1516
import Modal from './Modal';
@@ -26,6 +27,17 @@ function TestToolsModal() {
2627
const styles = useThemeStyles();
2728
const {translate} = useLocalize();
2829

30+
const shouldShowProfileTool = () => {
31+
const browser = getBrowser();
32+
const isSafariOrFirefox = browser === CONST.BROWSER.SAFARI || browser === CONST.BROWSER.FIREFOX;
33+
34+
if(isSafariOrFirefox || isChromeIOS()){
35+
return false;
36+
}
37+
38+
return true;
39+
};
40+
2941
return (
3042
<Modal
3143
isVisible={!!isTestToolsModalOpen}
@@ -39,7 +51,7 @@ function TestToolsModal() {
3951
>
4052
{translate('initialSettingsPage.troubleshoot.releaseOptions')}
4153
</Text>
42-
<ProfilingToolMenu />
54+
{shouldShowProfileTool() && <ProfilingToolMenu />}
4355
<ClientSideLoggingToolMenu />
4456
{!!shouldStoreLogs && (
4557
<TestToolRow title={translate('initialSettingsPage.troubleshoot.debugConsole')}>

0 commit comments

Comments
 (0)