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

Commit 386efae8 authored by Satakshi's avatar Satakshi Committed by Satakshi Rana
Browse files

Small Tweaks to Screenshot Intelligence log messages for better 

debugging.

Test: Device logcat
Bug: 145687348
Change-Id: I03c06c4ab7494899867a98df5d65000cfc2caccf
parent 4d4e91db

packages/SystemUI/src/com/android/systemui/screenshot/GlobalScreenshot.java

+10 −7
Original line number Diff line number Diff line
@@ -804,8 +804,8 @@ public class GlobalScreenshot { 
            List<Notification.Action> actions = smartActionsFuture.get(timeoutMs,

                    TimeUnit.MILLISECONDS);

            long waitTimeMs = SystemClock.uptimeMillis() - startTimeMs;

            Slog.d(TAG, String.format("Wait time for smart actions: %d ms",

                    waitTimeMs));

            Slog.d(TAG, String.format("Got %d smart actions. Wait time: %d ms",

                    actions.size(), waitTimeMs));

            notifyScreenshotOp(screenshotId, smartActionsProvider,

                    ScreenshotNotificationSmartActionsProvider.ScreenshotOp.WAIT_FOR_SMART_ACTIONS,

                    ScreenshotNotificationSmartActionsProvider.ScreenshotOpStatus.SUCCESS,
@@ -813,7 +813,8 @@ public class GlobalScreenshot { 
            return actions;

        } catch (Throwable e) {

            long waitTimeMs = SystemClock.uptimeMillis() - startTimeMs;

            Slog.d(TAG, "Failed to obtain screenshot notification smart actions.", e);

            Slog.e(TAG, String.format("Error getting smart actions. Wait time: %d ms", waitTimeMs),

                    e);

            ScreenshotNotificationSmartActionsProvider.ScreenshotOpStatus status =

                    (e instanceof TimeoutException)

                            ? ScreenshotNotificationSmartActionsProvider.ScreenshotOpStatus.TIMEOUT
@@ -942,14 +943,16 @@ public class GlobalScreenshot { 
    public static class SmartActionsReceiver extends BroadcastReceiver {

        @Override

        public void onReceive(Context context, Intent intent) {

            PendingIntent actionIntent = intent.getParcelableExtra(EXTRA_ACTION_INTENT);

            PendingIntent pendingIntent = intent.getParcelableExtra(EXTRA_ACTION_INTENT);

            Intent actionIntent = pendingIntent.getIntent();

            String actionType = intent.getStringExtra(EXTRA_ACTION_TYPE);

            Slog.d(TAG, "Executing smart action [" + actionType + "]:" + actionIntent);

            ActivityOptions opts = ActivityOptions.makeBasic();

            context.startActivityAsUser(actionIntent.getIntent(), opts.toBundle(),

            context.startActivityAsUser(actionIntent, opts.toBundle(),

                    UserHandle.CURRENT);



            Slog.d(TAG, "Screenshot notification smart action is invoked.");

            notifyScreenshotAction(context, intent.getStringExtra(EXTRA_ID),

                    intent.getStringExtra(EXTRA_ACTION_TYPE),

                    actionType,

                    true);

        }

    }

packages/SystemUI/src/com/android/systemui/screenshot/SaveImageInBackgroundTask.java

+3 −3
Original line number Diff line number Diff line
@@ -456,9 +456,9 @@ class SaveImageInBackgroundTask extends AsyncTask<Void, Void, Void> { 
                    SystemUiDeviceConfigFlags

                            .SCREENSHOT_NOTIFICATION_SMART_ACTIONS_TIMEOUT_MS,

                    1000);

            List<Notification.Action> smartActions = buildSmartActions(

                    GlobalScreenshot.getSmartActions(mScreenshotId, smartActionsFuture,

                            timeoutMs, mSmartActionsProvider), context);

            List<Notification.Action> smartActions = GlobalScreenshot.getSmartActions(mScreenshotId,

                    smartActionsFuture, timeoutMs, mSmartActionsProvider);

            smartActions = buildSmartActions(smartActions, context);

            for (Notification.Action action : smartActions) {

                notificationBuilder.addAction(action);

            }