Skip to content

Commit 5c959ba

Browse files
authored
Turn off new strings (#11285)
* Revert "VPN-7203 - Add a warning when not running with "unrestricted" background usage permission (#11169)" This reverts commit 94ea52f. * remove other 2 strings
1 parent 4d6617e commit 5c959ba

11 files changed

Lines changed: 0 additions & 221 deletions

File tree

android/AndroidManifest.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ file, You can obtain one at http://mozilla.org/MPL/2.0/.
1212
<uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED" />
1313
<uses-permission android:name="android.permission.QUERY_ALL_PACKAGES"/>
1414
<uses-permission android:name="android.permission.POST_NOTIFICATIONS"/>
15-
<uses-permission android:name="android.permission.REQUEST_IGNORE_BATTERY_OPTIMIZATIONS"/>
1615
<uses-permission android:name="com.google.android.gms.permission.AD_ID"/>
1716

1817
<!-- The following comment will be replaced upon deployment with default features based on the dependencies of the application.

android/common/src/main/java/org/mozilla/firefox/qt/common/BatteryOptimizationHelper.kt

Lines changed: 0 additions & 50 deletions
This file was deleted.

src/platforms/android/androidbatteryoptimizer.cpp

Lines changed: 0 additions & 43 deletions
This file was deleted.

src/platforms/android/androidbatteryoptimizer.h

Lines changed: 0 additions & 23 deletions
This file was deleted.

src/platforms/android/sources.cmake

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@ target_sources(shared-sources INTERFACE
77
${CMAKE_CURRENT_SOURCE_DIR}/platforms/android/androidcryptosettings.cpp
88
${CMAKE_CURRENT_SOURCE_DIR}/platforms/android/androidcommons.cpp
99
${CMAKE_CURRENT_SOURCE_DIR}/platforms/android/androidcommons.h
10-
${CMAKE_CURRENT_SOURCE_DIR}/platforms/android/androidbatteryoptimizer.h
11-
${CMAKE_CURRENT_SOURCE_DIR}/platforms/android/androidbatteryoptimizer.cpp
1210
)
1311

1412
target_link_libraries(shared-sources INTERFACE -ljnigraphics)

src/settingslist.h

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -325,16 +325,6 @@ SETTING_BOOL(onboardingCompleted, // getter
325325
false // sensitive (do not log)
326326
)
327327

328-
SETTING_BOOL(hasDismissedBatteryOptimization, // getter
329-
setHasDismissedBatteryOptimization, // setter
330-
removeHasDismissedBatteryOptimization, // remover
331-
hasHasDismissedBatteryOptimization, // has
332-
"hasDismissedBatteryOptimization", // key
333-
false, // default value
334-
true, // remove when reset (clears on logout)
335-
false // sensitive (do not log)
336-
)
337-
338328
SETTING_BOOL(onboardingDataCollectionEnabled, // getter
339329
setOnboardingDataCollectionEnabled, // setter
340330
removeOnboardingDataCollectionEnabled, // remover

src/translations/strings.yaml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -365,12 +365,6 @@ settings:
365365
dnsSettingsWarning: These options may cause some websites to break or not display content correctly.
366366
dnsSettingsInputPlaceholder: Enter custom DNS
367367
dnsSettingsDisconnectWarning: Changing these settings will cause the VPN to disconnect and reconnect automatically.
368-
batteryOptimizationWarning:
369-
value: On some devices, Mozilla VPN may be disconnected in the background. If you’ve experienced this, allow it to run in the background.
370-
comment: Warning shown in the settings screen on Android when battery optimization is enabled for the app.
371-
batteryOptimizationWarningButton:
372-
value: Allow background permissions
373-
comment: Button label for the call to action to allow background permissions on Android.
374368
appExclusionClearAllApps:
375369
value: Clear all
376370
comment: Button label to clear all apps from the exclusions list

src/ui/CMakeLists.txt

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -166,10 +166,7 @@ target_include_directories(mozillavpn-uiplugin PUBLIC
166166
${CMAKE_CURRENT_SOURCE_DIR}/types
167167
)
168168

169-
target_sources(mozillavpn-ui PRIVATE singletons/BatteryOptimizer.h)
170-
171169
if(${CMAKE_SYSTEM_NAME} STREQUAL "Android")
172-
target_compile_definitions(mozillavpn-ui PRIVATE MZ_ANDROID)
173170
target_sources(mozillavpn-ui PRIVATE singletons/VPNAndroidUtils.h)
174171
target_sources(mozillavpn-ui PRIVATE singletons/VPNAndroidCommons.h)
175172
endif()

src/ui/screens/home/servers/ServerList.qml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -220,11 +220,6 @@ FocusScope {
220220
return MZI18n.ServersViewRecommendedRefreshlLoadingLabel.arg(
221221
Math.round(VPNServerLatency.progress * 100));
222222

223-
case VPNServerLatency.Initial:
224-
return (VPNController.state === VPNController.StateOff)
225-
? MZI18n.ServersViewRecommendedRefreshNoUpdateLabel
226-
: MZI18n.ServersViewRecommendedRefreshNoUpdateDisabledLabel;
227-
228223
case VPNServerLatency.Loaded:
229224
default:
230225
const formatted = MZLocalizer.formatDate(

src/ui/screens/settings/ViewSettingsMenu.qml

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -22,30 +22,6 @@ MZViewBase {
2222
spacing: MZTheme.theme.windowMargin
2323
Layout.fillWidth: true
2424

25-
MZInformationCard {
26-
id: batteryOptimizationCard
27-
visible: !MZSettings.hasDismissedBatteryOptimization
28-
&& BatteryOptimizer.batteryOptimizationEnabled
29-
&& BatteryOptimizer.canTriggerIntent
30-
cardType: MZInformationCard.CardType.Warning
31-
Layout.preferredWidth: Math.min(window.width - MZTheme.theme.windowMargin * 2,
32-
MZTheme.theme.navBarMaxWidth)
33-
Layout.minimumHeight: batteryWarningText.height + MZTheme.theme.windowMargin * 2
34-
Layout.alignment: Qt.AlignHCenter
35-
36-
_infoContent: MZTextBlock {
37-
id: batteryWarningText
38-
Layout.fillWidth: true
39-
text: MZI18n.SettingsBatteryOptimizationWarning
40-
}
41-
_buttonAction: function() {
42-
MZSettings.hasDismissedBatteryOptimization = true
43-
BatteryOptimizer.triggerBatteryOptimizationIntent()
44-
}
45-
_buttonText : MZI18n.SettingsBatteryOptimizationWarningButton
46-
47-
}
48-
4925
ColumnLayout {
5026
Layout.fillWidth: true
5127
Layout.leftMargin: MZTheme.theme.windowMargin /2

0 commit comments

Comments
 (0)