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

Commit a6aff40f authored by Behnam Heydarshahi's avatar Behnam Heydarshahi Committed by Android (Google) Code Review
Browse files

Merge "QSTileLogger: use TileSpec for logbuffer map key" into main

parents 9bb446fa b6e7d1b8

packages/SystemUI/src/com/android/systemui/qs/tiles/base/logging/QSTileLogger.kt

+2 −2
Original line number Diff line number Diff line
@@ -33,7 +33,7 @@ import javax.inject.Inject 
class QSTileLogger

@Inject

constructor(

    @QSTilesLogBuffers logBuffers: Map<String, LogBuffer>,

    @QSTilesLogBuffers logBuffers: Map<TileSpec, LogBuffer>,

    private val factory: LogBufferFactory,

    private val mStatusBarStateController: StatusBarStateController,

) {
@@ -162,7 +162,7 @@ constructor( 


    private fun TileSpec.getLogBuffer(): LogBuffer =

        synchronized(logBufferCache) {

            logBufferCache.getOrPut(this.spec) {

            logBufferCache.getOrPut(this) {

                factory.create(

                    this.getLogTag(),

                    BUFFER_MAX_SIZE /* maxSize */,

packages/SystemUI/tests/src/com/android/systemui/qs/tiles/base/logging/QSTileLoggerTest.kt

+2 −5
Original line number Diff line number Diff line
@@ -55,12 +55,9 @@ class QSTileLoggerTest : SysuiTestCase() { 
    fun setup() {

        MockitoAnnotations.initMocks(this)

        whenever(logBufferFactory.create(any(), any(), any())).thenReturn(logBuffer)

        val tileSpec: TileSpec = TileSpec.create("chatty_tile")

        underTest =

            QSTileLogger(

                mapOf("chatty_tile" to chattyLogBuffer),

                logBufferFactory,

                statusBarController

            )

            QSTileLogger(mapOf(tileSpec to chattyLogBuffer), logBufferFactory, statusBarController)

    }



    @Test