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

Commit 494e9f8a authored by Pechetty Sravani (xWF)'s avatar Pechetty Sravani (xWF) Committed by Android (Google) Code Review
Browse files

Revert "Use the expected frame duration from SF instead of HWUI." 

Revert submission 28899062-frame_duration

Reason for revert: Droid monitor created revert due to this b/362835127.

Reverted changes: /q/submissionid:28899062-frame_duration

Change-Id: I969312c37cf9836b95b86106aa6cd80aa395f043
parent 1a1427d1

AconfigFlags.bp

+0 −15
Original line number Diff line number Diff line
@@ -100,7 +100,6 @@ aconfig_declarations_group { 
        "framework-jobscheduler-job.flags-aconfig-java",

        "framework_graphics_flags_java_lib",

        "hwui_flags_java_lib",

        "interaction_jank_monitor_flags_lib",

        "libcore_exported_aconfig_flags_lib",

        "libgui_flags_java_lib",

        "power_flags_lib",
@@ -1566,17 +1565,3 @@ java_aconfig_library { 
    aconfig_declarations: "dropbox_flags",

    defaults: ["framework-minus-apex-aconfig-java-defaults"],

}
 


// Zero Jank

aconfig_declarations {

    name: "interaction_jank_monitor_flags",

    package: "com.android.internal.jank",

    container: "system",

    srcs: ["core/java/com/android/internal/jank/flags.aconfig"],

}



java_aconfig_library {

    name: "interaction_jank_monitor_flags_lib",

    aconfig_declarations: "interaction_jank_monitor_flags",

    defaults: ["framework-minus-apex-aconfig-java-defaults"],

}

core/java/com/android/internal/jank/FrameTracker.java

+1 −6
Original line number Diff line number Diff line
@@ -158,17 +158,12 @@ public class FrameTracker implements HardwareRendererObserver.OnFrameMetricsAvai 
            this.surfaceControlCallbackFired = true;

            this.jankType = jankStat.jankType;

            this.refreshRate = DisplayRefreshRate.getRefreshRate(jankStat.frameIntervalNs);

            if (Flags.useSfFrameDuration()) {

                this.totalDurationNanos = jankStat.actualAppFrameTimeNs;

            }

            return this;

        }



        private JankInfo update(long totalDurationNanos, boolean isFirstFrame) {

            this.hwuiCallbackFired = true;

            if (!Flags.useSfFrameDuration()) {

            this.totalDurationNanos = totalDurationNanos;

            }

            this.isFirstFrame = isFirstFrame;

            return this;

        }

core/java/com/android/internal/jank/flags.aconfig

deleted100644 → 0
+0 −9
Original line number Diff line number Diff line 
package: "com.android.internal.jank"

container: "system"



flag {

  name: "use_sf_frame_duration"

  namespace: "android_platform_window_surfaces"

  description: "Whether to get the frame duration from SurfaceFlinger, or HWUI"

  bug: "354763298"

}