Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7632298b authored by Evan Laird's avatar Evan Laird
Browse files

Remove top level interface implementation from PrivacyDotViewController

addresses code comments

Test: builds
Change-Id: Ife29828421e2009fe1c98d0b8a4c35f6aaf2cdee
parent d779288d
Loading
Loading
Loading
Loading
+8 −7
Original line number Diff line number Diff line
@@ -70,7 +70,7 @@ class PrivacyDotViewController @Inject constructor(
    private val configurationController: ConfigurationController,
    private val contentInsetsProvider: StatusBarContentInsetsProvider,
    private val animationScheduler: SystemStatusAnimationScheduler
) : StatusBarContentInsetsChangedListener {
) {
    private var sbHeightPortrait = 0
    private var sbHeightLandscape = 0

@@ -98,7 +98,13 @@ class PrivacyDotViewController @Inject constructor(
        get() = if (!this::tl.isInitialized) sequenceOf() else sequenceOf(tl, tr, br, bl)

    init {
        contentInsetsProvider.addCallback(this)
        contentInsetsProvider.addCallback(object : StatusBarContentInsetsChangedListener {
            override fun onStatusBarContentInsetsChanged() {
                dlog("onStatusBarContentInsetsChanged: ")
                setNewLayoutRects()
            }
        })

        configurationController.addCallback(object : ConfigurationController.ConfigurationListener {
            override fun onLayoutDirectionChanged(isRtl: Boolean) {
                synchronized(this) {
@@ -533,11 +539,6 @@ class PrivacyDotViewController @Inject constructor(
        return -1
    }

    override fun onStatusBarContentInsetsChanged() {
        Log.d(TAG, "onStatusBarContentInsetsChanged: ")
        setNewLayoutRects()
    }

    // Returns [left, top, right, bottom] aka [seascape, none, landscape, upside-down]
    private fun getLayoutRects(): List<Rect> {
        val left = contentInsetsProvider.getStatusBarContentInsetsForRotation(ROTATION_SEASCAPE)