Skip to content

Commit e220c72

Browse files
committed
Uplift of #20281 (squashed) to beta
1 parent 0a87770 commit e220c72

File tree

2 files changed

+7
-6
lines changed

2 files changed

+7
-6
lines changed

android/java/org/chromium/chrome/browser/appmenu/BraveTabbedAppMenuPropertiesDelegate.java

+1-3
Original file line numberDiff line numberDiff line change
@@ -238,9 +238,7 @@ public void onMenuDismissed() {
238238
mMenu.removeItem(R.id.brave_playlist_id);
239239
mMenu.removeItem(R.id.brave_speedreader_id);
240240
mMenu.removeItem(R.id.exit_id);
241-
if (BraveVpnUtils.isVpnFeatureSupported(mContext)) {
242-
mMenu.removeItem(R.id.request_brave_vpn_row_menu_id);
243-
}
241+
mMenu.removeItem(R.id.request_brave_vpn_row_menu_id);
244242
}
245243

246244
@Override

android/java/org/chromium/chrome/browser/settings/BraveMainPreferencesBase.java

+6-3
Original file line numberDiff line numberDiff line change
@@ -248,9 +248,11 @@ public <T extends Preference> T findPreference(CharSequence key) {
248248
private void rearrangePreferenceOrders() {
249249
int firstSectionOrder = 0;
250250

251-
if (BraveVpnPrefUtils.shouldShowCallout() && !BraveVpnPrefUtils.isSubscriptionPurchase()
251+
if (getActivity() != null && !getActivity().isFinishing()
252+
&& BraveVpnPrefUtils.shouldShowCallout()
253+
&& !BraveVpnPrefUtils.isSubscriptionPurchase()
252254
&& BraveVpnUtils.isVpnFeatureSupported(getActivity())) {
253-
if (getActivity() != null && mVpnCalloutPreference == null) {
255+
if (mVpnCalloutPreference == null) {
254256
mVpnCalloutPreference = new VpnCalloutPreference(getActivity());
255257
}
256258
if (mVpnCalloutPreference != null) {
@@ -277,7 +279,8 @@ private void rearrangePreferenceOrders() {
277279
removePreferenceIfPresent(PREF_BRAVE_PLAYLIST);
278280
}
279281

280-
if (BraveVpnUtils.isVpnFeatureSupported(getActivity())) {
282+
if (getActivity() != null && !getActivity().isFinishing()
283+
&& BraveVpnUtils.isVpnFeatureSupported(getActivity())) {
281284
findPreference(PREF_BRAVE_VPN).setOrder(++firstSectionOrder);
282285
} else {
283286
removePreferenceIfPresent(PREF_BRAVE_VPN);

0 commit comments

Comments
 (0)