Unverified Commit 44a5ed86 authored by Matthew Finkel's avatar Matthew Finkel Committed by boklm
Browse files

Add Security Level UI

Bug 40026: Implement Security Level settings

Bug 40026: Integrate Security Level settings
parent 34fcee15
Loading
Loading
Loading
Loading
+1 −0
Original line number Diff line number Diff line
@@ -113,6 +113,7 @@ class Core(
                context,
                R.color.foundation_normal_theme
            ),
            torSecurityLevel = context.settings().torSecurityLevel().intRepresentation,
            spoofEnglish = context.settings().spoofEnglish
        )

+2 −0
Original line number Diff line number Diff line
@@ -499,6 +499,8 @@ class DefaultSessionControlController(
    }

    override fun handleOpenSecurityLevelSettingsClicked() {
        val directions = HomeFragmentDirections.actionGlobalTorSecurityLevelFragment()
        navController.nav(R.id.homeFragment, directions)
    }

    override fun handleWhatsNewGetAnswersClicked() {
+35 −10
Original line number Diff line number Diff line
@@ -8,8 +8,11 @@ import android.view.View
import androidx.recyclerview.widget.RecyclerView
import kotlinx.android.synthetic.main.tor_onboarding_security_level.view.*
import org.mozilla.fenix.R
import org.mozilla.fenix.ext.components
import org.mozilla.fenix.home.sessioncontrol.OnboardingInteractor
import org.mozilla.fenix.onboarding.OnboardingRadioButton
import org.mozilla.fenix.tor.SecurityLevel
import org.mozilla.fenix.tor.SecurityLevelUtil
import org.mozilla.fenix.utils.view.addToRadioGroup

class TorOnboardingSecurityLevelViewHolder(
@@ -33,34 +36,56 @@ class TorOnboardingSecurityLevelViewHolder(
        )

        view.open_settings_button.setOnClickListener {
            interactor.onOpenSettingsClicked()
            interactor.onOpenSecurityLevelSettingsClicked()
        }

        setupRadioGroup()
        setupRadioGroup(view)
    }

    private fun setupRadioGroup() {
    private fun setupRadioGroup(view: View) {

        addToRadioGroup(standardSecurityLevel, saferSecurityLevel, safestSecurityLevel)

        standardSecurityLevel.isChecked = true
        safestSecurityLevel.isChecked = false
        saferSecurityLevel.isChecked = false
        val securityLevel = try {
            SecurityLevelUtil.getSecurityLevelFromInt(
                view.context.components.core.engine.settings.torSecurityLevel
            )
        } catch (e: IllegalStateException) {
            SecurityLevel.STANDARD
        }

        standardSecurityLevel.isChecked = securityLevel == SecurityLevel.STANDARD
        safestSecurityLevel.isChecked = securityLevel == SecurityLevel.SAFEST
        saferSecurityLevel.isChecked = securityLevel == SecurityLevel.SAFER

        standardSecurityLevel.onClickListener {
            updateSecurityLevel()
            updateSecurityLevel(SecurityLevel.STANDARD)
        }

        saferSecurityLevel.onClickListener {
            updateSecurityLevel()
            updateSecurityLevel(SecurityLevel.SAFER)
        }

        safestSecurityLevel.onClickListener {
            updateSecurityLevel()
            updateSecurityLevel(SecurityLevel.SAFEST)
        }

        updateSecurityLevel(securityLevel)
    }

    private fun updateSecurityLevel() {
    private fun updateSecurityLevel(newLevel: SecurityLevel) {
        val resources = itemView.context.resources
        val securityLevel = when (newLevel) {
            SecurityLevel.STANDARD -> resources.getString(R.string.tor_security_level_standard_option)
            SecurityLevel.SAFER -> resources.getString(R.string.tor_security_level_safer_option)
            SecurityLevel.SAFEST -> resources.getString(R.string.tor_security_level_safest_option)
        }
        itemView.current_level.text = resources.getString(
            R.string.tor_onboarding_chosen_security_level_label, securityLevel
        )
        itemView.context.components.let {
            it.core.engine.settings.torSecurityLevel = newLevel.intRepresentation
        }
    }

    companion object {
+3 −0
Original line number Diff line number Diff line
@@ -254,6 +254,9 @@ class SettingsFragment : PreferenceFragmentCompat() {
            resources.getString(R.string.pref_key_tor_network_settings) -> {
                SettingsFragmentDirections.actionSettingsFragmentToTorNetworkSettingsFragment()
            }
            resources.getString(R.string.pref_key_tor_security_level_settings) -> {
                SettingsFragmentDirections.actionSettingsFragmentToTorSecurityLevelFragment()
            }
            resources.getString(R.string.pref_key_tracking_protection_settings) -> {
                requireContext().metrics.track(Event.TrackingProtectionSettings)
                SettingsFragmentDirections.actionSettingsFragmentToTrackingProtectionFragment()
+86 −0
Original line number Diff line number Diff line
/* This Source Code Form is subject to the terms of the Mozilla Public
 * License, v. 2.0. If a copy of the MPL was not distributed with this
 * file, You can obtain one at http://mozilla.org/MPL/2.0/. */

package org.mozilla.fenix.settings

import android.os.Bundle
import androidx.preference.PreferenceFragmentCompat
import org.mozilla.fenix.R
import org.mozilla.fenix.ext.components
import org.mozilla.fenix.ext.settings
import org.mozilla.fenix.ext.showToolbar
import org.mozilla.fenix.tor.SecurityLevel
import org.mozilla.fenix.tor.SecurityLevelUtil
import org.mozilla.fenix.utils.view.GroupableRadioButton
import org.mozilla.fenix.utils.view.addToRadioGroup
import org.mozilla.fenix.utils.view.uncheckAll

/**
 * Lets the user choose their security level
 */
@Suppress("SpreadOperator")
class TorSecurityLevelFragment : PreferenceFragmentCompat() {
    private val securityLevelRadioGroups = mutableListOf<GroupableRadioButton>()
    private var previousSecurityLevel: SecurityLevel? = null

    override fun onResume() {
        super.onResume()
        showToolbar(getString(R.string.preferences_tor_security_level_options))
    }

    override fun onCreatePreferences(savedInstanceState: Bundle?, rootKey: String?) {
        setPreferencesFromResource(R.xml.tor_security_level_preferences, rootKey)

        val currentLevel: SecurityLevel? = context?.components?.let {
            try {
                SecurityLevelUtil.getSecurityLevelFromInt(
                    it.core.engine.settings.torSecurityLevel
                )
            } catch (e: IllegalStateException) {
                // The default state is 0. If we get an invalid state then
                // default to Standard (4).
                SecurityLevel.STANDARD
            }
        }

        if (currentLevel == null) {
            throw IllegalStateException("context or Components is null.")
        }

        val radioSafer = bindSecurityLevelRadio(SecurityLevel.SAFER)
        val radioSafest = bindSecurityLevelRadio(SecurityLevel.SAFEST)
        val radioStandard = bindSecurityLevelRadio(SecurityLevel.STANDARD)

        securityLevelRadioGroups.addAll(mutableListOf(radioSafer, radioSafest, radioStandard))
        // `*` is Kotlin's "spread" operator, for expanding an Array as a vararg.
        addToRadioGroup(*securityLevelRadioGroups.toTypedArray())

        securityLevelRadioGroups.uncheckAll()
        val securityLevelRadioButton = requirePreference<RadioButtonPreference>(currentLevel.preferenceKey)
        // Cache this for later comparison in the OnPreferenceChangeListener
        previousSecurityLevel = currentLevel
        securityLevelRadioButton.setCheckedWithoutClickListener(true)
    }

    private fun bindSecurityLevelRadio(
        level: SecurityLevel
    ): RadioButtonPreference {
        val radio = requirePreference<RadioButtonPreference>(level.preferenceKey)

        radio.summary = getString(level.contentDescriptionRes)

        radio.apply {
            setOnPreferenceChangeListener<Boolean> { preference, isChecked ->
                if (isChecked && (previousSecurityLevel!! != level)) {
                    preference.context.components.core.engine.settings.torSecurityLevel =
                        level.intRepresentation
                    previousSecurityLevel = level
                }
                true
            }
        }

        return radio
    }
}
Loading