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

Commit 6a3b93c3 authored by Colin Cross's avatar Colin Cross Committed by Cherrypicker Worker
Browse files

Fix kotlin nullable errors in frameworks/libs/systemui 

Fix kotlin nullable errors that were exposed by setting the retention
of android.annotation.NonNull and android.annotation.Nullable to
class retention.

Bug: 294110802
Test: builds
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:b80daead91deb31ba2cd8b7577f89e697a0301fa)
Merged-In: If8dc1d32d7fd02e41c12cfb1571374c1e974e87c
Change-Id: If8dc1d32d7fd02e41c12cfb1571374c1e974e87c
parent 201db258

motiontoollib/tests/com/android/app/motiontool/DdmHandleMotionToolTest.kt

+4 −1
Original line number Diff line number Diff line
@@ -130,7 +130,10 @@ class DdmHandleMotionToolTest { 
            val traceId = beginTraceResponse.beginTrace.traceId



            Choreographer.getInstance().postFrameCallback {

                activity.findViewById<View>(android.R.id.content).viewTreeObserver.dispatchOnDraw()

                activity

                    .requireViewById<View>(android.R.id.content)

                    .viewTreeObserver

                    .dispatchOnDraw()



                val pollTraceResponse = performPollTraceRequest(traceId)

                assertEquals(1, pollTraceResponse.pollTrace.data.frameDataList.size)

motiontoollib/tests/com/android/app/motiontool/MotionToolManagerTest.kt

+4 −1
Original line number Diff line number Diff line
@@ -85,7 +85,10 @@ class MotionToolManagerTest { 
        activityScenarioRule.scenario.onActivity { activity ->

            val traceId = motionToolManager.beginTrace(getActivityViewRootId())

            Choreographer.getInstance().postFrameCallback {

                activity.findViewById<View>(android.R.id.content).viewTreeObserver.dispatchOnDraw()

                activity

                    .requireViewById<View>(android.R.id.content)

                    .viewTreeObserver

                    .dispatchOnDraw()



                val polledData = motionToolManager.pollTrace(traceId)

                assertEquals(1, polledData.frameDataList.size)

viewcapturelib/tests/com/android/app/viewcapture/SettingsAwareViewCaptureTest.kt

+20 −6
Original line number Diff line number Diff line
@@ -52,14 +52,21 @@ class SettingsAwareViewCaptureTest { 
        activityScenarioRule.scenario.onActivity { activity ->

            val viewCapture: ViewCapture =

                SettingsAwareViewCapture(context, Choreographer.getInstance(), MAIN_EXECUTOR)

            val rootView: View = activity.findViewById(android.R.id.content)

            val rootView: View = activity.requireViewById(android.R.id.content)



            val closeable: SafeCloseable = viewCapture.startCapture(rootView, "rootViewId")

            Choreographer.getInstance().postFrameCallback {

                rootView.viewTreeObserver.dispatchOnDraw()



                assertEquals(0, viewCapture.getDumpTask(

                        activity.findViewById(android.R.id.content)).get().get().frameDataList.size)

                assertEquals(

                    0,

                    viewCapture

                        .getDumpTask(activity.requireViewById(android.R.id.content))

                        .get()

                        .get()

                        .frameDataList

                        .size

                )

                closeable.close()

            }

        }
@@ -72,14 +79,21 @@ class SettingsAwareViewCaptureTest { 
        activityScenarioRule.scenario.onActivity { activity ->

            val viewCapture: ViewCapture =

                SettingsAwareViewCapture(context, Choreographer.getInstance(), MAIN_EXECUTOR)

            val rootView: View = activity.findViewById(android.R.id.content)

            val rootView: View = activity.requireViewById(android.R.id.content)



            val closeable: SafeCloseable = viewCapture.startCapture(rootView, "rootViewId")

            Choreographer.getInstance().postFrameCallback {

                rootView.viewTreeObserver.dispatchOnDraw()



                assertEquals(1, viewCapture.getDumpTask(activity.findViewById(

                        android.R.id.content)).get().get().frameDataList.size)

                assertEquals(

                    1,

                    viewCapture

                        .getDumpTask(activity.requireViewById(android.R.id.content))

                        .get()

                        .get()

                        .frameDataList

                        .size

                )



                closeable.close()

            }

viewcapturelib/tests/com/android/app/viewcapture/ViewCaptureTest.kt

+3 −3
Original line number Diff line number Diff line
@@ -54,7 +54,7 @@ class ViewCaptureTest { 
        activityScenarioRule.scenario.onActivity { activity ->

            Choreographer.getInstance().postFrameCallback {

                val closeable = startViewCaptureAndInvalidateNTimes(1, activity)

                val rootView = activity.findViewById<View>(android.R.id.content)

                val rootView = activity.requireViewById<View>(android.R.id.content)

                val data = viewCapture.getDumpTask(rootView).get().get()



                assertEquals(1, data.frameDataList.size)
@@ -69,7 +69,7 @@ class ViewCaptureTest { 
        activityScenarioRule.scenario.onActivity { activity ->

            Choreographer.getInstance().postFrameCallback {

                val closeable = startViewCaptureAndInvalidateNTimes(memorySize + 5, activity)

                val rootView = activity.findViewById<View>(android.R.id.content)

                val rootView = activity.requireViewById<View>(android.R.id.content)

                val data = viewCapture.getDumpTask(rootView).get().get()



                // since ViewCapture MEMORY_SIZE is [viewCaptureMemorySize], only
@@ -83,7 +83,7 @@ class ViewCaptureTest { 
    }



    private fun startViewCaptureAndInvalidateNTimes(n: Int, activity: TestActivity): SafeCloseable {

        val rootView: View = activity.findViewById(android.R.id.content)

        val rootView: View = activity.requireViewById(android.R.id.content)

        val closeable: SafeCloseable = viewCapture.startCapture(rootView, "rootViewId")

        dispatchOnDraw(rootView, times = n)

        return closeable