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

Commit f9969f70 authored by Kean Mariotti's avatar Kean Mariotti
Browse files

Revert^2 Adapt flicker tests to perfetto-based SurfaceFlinger tracing 

- Change permissions to allow connection to perfetto trace processor
- Add flickerlib-trace_processor_shell dependency to pull trace_processor_shell
  prebuilts into the apk
- Adapt test code to new perfetto trace monitor
- Remove obsolete TRACE_CRITICAL layer tracing flag in test code

Bug: b/284424740
Test: atest FlickerTestsAppClose && \
      atest FlickerTestsIme && \
      atest FlickerTestsQuickswitch && \
      atest FlickerTestsAppLaunch && \
      atest FlickerTestsOther && \
      atest FlickerTestsRotation && \
      atest TaskOrganizerTest
Change-Id: I9d3e645ad387ebe74ea4884d647dccab07415831
parent 6f43888d

libs/WindowManager/Shell/tests/flicker/Android.bp

+1 −0
Original line number Diff line number Diff line
@@ -126,6 +126,7 @@ java_defaults { 
        "flickertestapplib",

        "flickerlib",

        "flickerlib-helpers",

        "flickerlib-trace_processor_shell",

        "platform-test-annotations",

        "wm-flicker-common-app-helpers",

        "wm-flicker-common-assertions",

libs/WindowManager/Shell/tests/flicker/manifests/AndroidManifest.xml

+6 −2
Original line number Diff line number Diff line
@@ -45,9 +45,13 @@ 
    <uses-permission android:name="android.permission.MANAGE_ACTIVITY_TASKS" />

    <!-- Enable bubble notification-->

    <uses-permission android:name="android.permission.STATUS_BAR_SERVICE" />

    <!-- Allow the test to connect to perfetto trace processor -->

    <uses-permission android:name="android.permission.INTERNET"/>



    <!-- Allow the test to write directly to /sdcard/ -->

    <application android:requestLegacyExternalStorage="true" android:largeHeap="true">

    <!-- Allow the test to write directly to /sdcard/ and connect to trace processor -->

    <application android:requestLegacyExternalStorage="true"

                 android:networkSecurityConfig="@xml/network_security_config"

                 android:largeHeap="true">

        <uses-library android:name="android.test.runner"/>



        <service android:name=".NotificationListener"

libs/WindowManager/Shell/tests/flicker/res/xml/network_security_config.xml

0 → 100644
+22 −0
Original line number Diff line number Diff line 
<?xml version="1.0" encoding="utf-8"?>

<!--

  ~ Copyright (C) 2023 The Android Open Source Project

  ~

  ~ Licensed under the Apache License, Version 2.0 (the "License");

  ~ you may not use this file except in compliance with the License.

  ~ You may obtain a copy of the License at

  ~

  ~      http://www.apache.org/licenses/LICENSE-2.0

  ~

  ~ Unless required by applicable law or agreed to in writing, software

  ~ distributed under the License is distributed on an "AS IS" BASIS,

  ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

  ~ See the License for the specific language governing permissions and

  ~ limitations under the License.

  -->



<network-security-config>

    <domain-config cleartextTrafficPermitted="true">

        <domain includeSubdomains="true">localhost</domain>

    </domain-config>

</network-security-config>

tests/FlickerTests/Android.bp

+1 −0
Original line number Diff line number Diff line
@@ -95,6 +95,7 @@ java_defaults { 
        "flickertestapplib",

        "flickerlib",

        "flickerlib-helpers",

        "flickerlib-trace_processor_shell",

        "platform-test-annotations",

        "wm-flicker-common-app-helpers",

        "wm-shell-flicker-utils",

tests/FlickerTests/manifests/AndroidManifest.xml

+6 −2
Original line number Diff line number Diff line
@@ -44,8 +44,12 @@ 
    <uses-permission android:name="android.permission.MANAGE_ACTIVITY_TASKS" />

    <!-- ActivityOptions.makeCustomTaskAnimation() -->

    <uses-permission android:name="android.permission.START_TASKS_FROM_RECENTS" />

    <!-- Allow the test to write directly to /sdcard/ -->

    <application android:requestLegacyExternalStorage="true" android:largeHeap="true">

    <!-- Allow the test to connect to perfetto trace processor -->

    <uses-permission android:name="android.permission.INTERNET"/>

    <!-- Allow the test to write directly to /sdcard/ and connect to trace processor -->

    <application android:requestLegacyExternalStorage="true"

                 android:networkSecurityConfig="@xml/network_security_config"

                 android:largeHeap="true">

        <uses-library android:name="android.test.runner"/>

        <uses-library android:name="androidx.window.extensions" android:required="false"/>