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

Commit 40abed43 authored by Yurii Zubrytskyi's avatar Yurii Zubrytskyi Committed by Android (Google) Code Review
Browse files

Revert "[res] Allow changing RRO state that target android" 

Revert submission 30159301-rro_enable_android

Reason for revert: not needed after the test is configured correctly (ag/30491922)

Reverted changes: /q/submissionid:30159301-rro_enable_android

Change-Id: If3ca2384f551a7bf38456d52759e1a8d243d42a3
parent 78f5213f

services/core/java/com/android/server/om/OverlayActorEnforcer.java

+2 −7
Original line number Diff line number Diff line
@@ -19,7 +19,6 @@ package com.android.server.om; 
import android.annotation.NonNull;

import android.content.om.OverlayInfo;

import android.content.om.OverlayableInfo;

import android.content.res.Flags;

import android.net.Uri;

import android.os.Process;

import android.text.TextUtils;
@@ -163,15 +162,11 @@ public class OverlayActorEnforcer { 
            return ActorState.UNABLE_TO_GET_TARGET_OVERLAYABLE;

        }



        // Framework doesn't have <overlayable> declaration by design, and we still want to be able

        // to enable its overlays from the packages with the permission.

        if (targetOverlayable == null

                && !(Flags.rroControlForAndroidNoOverlayable() && targetPackageName.equals(

                "android"))) {

        if (targetOverlayable == null) {

            return ActorState.MISSING_OVERLAYABLE;

        }



        final String actor = targetOverlayable == null ? null : targetOverlayable.actor;

        String actor = targetOverlayable.actor;

        if (TextUtils.isEmpty(actor)) {

            // If there's no actor defined, fallback to the legacy permission check

            try {