diff options
| author | 2024-08-01 17:49:12 +0000 | |
|---|---|---|
| committer | 2024-08-01 17:49:12 +0000 | |
| commit | f0c9c4d68c3e64be05dd9fcb449e1437c4627a9a (patch) | |
| tree | ae9c3e063879c2dd00f5e31afaa775340d449985 | |
| parent | 215452e2dbb8b9c4686ed156f873899a5dabb319 (diff) | |
| parent | 93ab91a21c90a66e58a6aaea0b5fbb400c8333e5 (diff) | |
Merge "Remove screenshot_private_profile_accessibility_announcement_fix" into main
3 files changed, 8 insertions, 38 deletions
diff --git a/packages/SystemUI/aconfig/systemui.aconfig b/packages/SystemUI/aconfig/systemui.aconfig index 49098f360a4b..f2c4a7fe42eb 100644 --- a/packages/SystemUI/aconfig/systemui.aconfig +++ b/packages/SystemUI/aconfig/systemui.aconfig @@ -589,16 +589,6 @@ flag { } flag { - name: "screenshot_private_profile_accessibility_announcement_fix" - namespace: "systemui" - description: "Modified a11y announcement for private space screenshots" - bug: "326941376" - metadata { - purpose: PURPOSE_BUGFIX - } -} - -flag { name: "screenshot_private_profile_behavior_fix" namespace: "systemui" description: "Private profile support for screenshots" diff --git a/packages/SystemUI/src/com/android/systemui/screenshot/LegacyScreenshotController.java b/packages/SystemUI/src/com/android/systemui/screenshot/LegacyScreenshotController.java index bc8642c67fb4..a77375c14f26 100644 --- a/packages/SystemUI/src/com/android/systemui/screenshot/LegacyScreenshotController.java +++ b/packages/SystemUI/src/com/android/systemui/screenshot/LegacyScreenshotController.java @@ -19,7 +19,6 @@ package com.android.systemui.screenshot; import static android.content.res.Configuration.ORIENTATION_PORTRAIT; import static android.view.WindowManager.LayoutParams.TYPE_SCREENSHOT; -import static com.android.systemui.Flags.screenshotPrivateProfileAccessibilityAnnouncementFix; import static com.android.systemui.Flags.screenshotSaveImageExporter; import static com.android.systemui.screenshot.LogConfig.DEBUG_ANIM; import static com.android.systemui.screenshot.LogConfig.DEBUG_CALLBACK; @@ -355,19 +354,9 @@ public class LegacyScreenshotController implements InteractiveScreenshotHandler void prepareViewForNewScreenshot(@NonNull ScreenshotData screenshot, String oldPackageName) { withWindowAttached(() -> { - if (screenshotPrivateProfileAccessibilityAnnouncementFix()) { - mAnnouncementResolver.getScreenshotAnnouncement( - screenshot.getUserHandle().getIdentifier(), - mViewProxy::announceForAccessibility); - } else { - if (mUserManager.isManagedProfile(screenshot.getUserHandle().getIdentifier())) { - mViewProxy.announceForAccessibility(mContext.getResources().getString( - R.string.screenshot_saving_work_profile_title)); - } else { - mViewProxy.announceForAccessibility( - mContext.getResources().getString(R.string.screenshot_saving_title)); - } - } + mAnnouncementResolver.getScreenshotAnnouncement( + screenshot.getUserHandle().getIdentifier(), + mViewProxy::announceForAccessibility); }); mViewProxy.reset(); diff --git a/packages/SystemUI/src/com/android/systemui/screenshot/ScreenshotController.java b/packages/SystemUI/src/com/android/systemui/screenshot/ScreenshotController.java index ec529cd6fa52..540d4c43c58d 100644 --- a/packages/SystemUI/src/com/android/systemui/screenshot/ScreenshotController.java +++ b/packages/SystemUI/src/com/android/systemui/screenshot/ScreenshotController.java @@ -19,7 +19,6 @@ package com.android.systemui.screenshot; import static android.content.res.Configuration.ORIENTATION_PORTRAIT; import static android.view.WindowManager.LayoutParams.TYPE_SCREENSHOT; -import static com.android.systemui.Flags.screenshotPrivateProfileAccessibilityAnnouncementFix; import static com.android.systemui.Flags.screenshotSaveImageExporter; import static com.android.systemui.screenshot.LogConfig.DEBUG_ANIM; import static com.android.systemui.screenshot.LogConfig.DEBUG_CALLBACK; @@ -355,19 +354,11 @@ public class ScreenshotController implements InteractiveScreenshotHandler { void prepareViewForNewScreenshot(@NonNull ScreenshotData screenshot, String oldPackageName) { withWindowAttached(() -> { - if (screenshotPrivateProfileAccessibilityAnnouncementFix()) { - mAnnouncementResolver.getScreenshotAnnouncement( - screenshot.getUserHandle().getIdentifier(), - mViewProxy::announceForAccessibility); - } else { - if (mUserManager.isManagedProfile(screenshot.getUserHandle().getIdentifier())) { - mViewProxy.announceForAccessibility(mContext.getResources().getString( - R.string.screenshot_saving_work_profile_title)); - } else { - mViewProxy.announceForAccessibility( - mContext.getResources().getString(R.string.screenshot_saving_title)); - } - } + mAnnouncementResolver.getScreenshotAnnouncement( + screenshot.getUserHandle().getIdentifier(), + announcement -> { + mViewProxy.announceForAccessibility(announcement); + }); }); mViewProxy.reset(); |