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

Commit 04277913 authored by Perry Wu's avatar Perry Wu
Browse files

Add logs for updating PIP remote actions 

Adds additional logging for when we update remote actions.

Flag: EXEMPT bugfix
Test: Manually check logs
Bug: b/349872888
Change-Id: Ie85406f80cd790aa8cea96564a720f562d7c5183
parent eeb12637

libs/WindowManager/Shell/src/com/android/wm/shell/pip/PipTaskOrganizer.java

+13 −0
Original line number Diff line number Diff line
@@ -99,6 +99,7 @@ import java.io.PrintWriter; 
import java.lang.ref.WeakReference;

import java.util.Objects;

import java.util.Optional;

import java.util.StringJoiner;

import java.util.function.Consumer;

import java.util.function.IntConsumer;
@@ -831,6 +832,7 @@ public class PipTaskOrganizer implements ShellTaskOrganizer.TaskListener, 
                    mPictureInPictureParams.getTitle());

            mPipParamsChangedForwarder.notifySubtitleChanged(

                    mPictureInPictureParams.getSubtitle());

            logRemoteActions(mPictureInPictureParams);

        }



        mPipUiEventLoggerLogger.setTaskInfo(mTaskInfo);
@@ -1112,6 +1114,7 @@ public class PipTaskOrganizer implements ShellTaskOrganizer.TaskListener, 
        }

        applyNewPictureInPictureParams(newParams);

        mPictureInPictureParams = newParams;

        logRemoteActions(mPictureInPictureParams);

    }



    @Override
@@ -1420,6 +1423,16 @@ public class PipTaskOrganizer implements ShellTaskOrganizer.TaskListener, 
        }

    }



    private void logRemoteActions(@NonNull PictureInPictureParams params) {

        StringJoiner sj = new StringJoiner("|", "[", "]");

        if (params.hasSetActions()) {

            params.getActions().forEach((action) -> sj.add(action.getTitle()));

        }



        ProtoLog.d(ShellProtoLogGroup.WM_SHELL_PICTURE_IN_PICTURE,

                "%s: PIP remote actions=%s", TAG, sj.toString());

    }



    /**

     * Animates resizing of the pinned stack given the duration.

     */