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

Commit c3dc1c1b authored by Matt Casey's avatar Matt Casey Committed by Automerger Merge Worker
Browse files

Merge "Remove SCREENSHOT_WORK_PROFILE_POLICY" into udc-dev am: 6ee67bcd am: a99d3159 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/22144274



Change-Id: Id91aafaf196de57b0310bf339076a47c75048352
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 99b28114 a99d3159

packages/SystemUI/src/com/android/systemui/flags/Flags.kt

+0 −4
Original line number Original line Diff line number Diff line
@@ -569,10 +569,6 @@ object Flags { 
    val TRACKPAD_GESTURE_COMMON = releasedFlag(1210, "trackpad_gesture_common")

    val TRACKPAD_GESTURE_COMMON = releasedFlag(1210, "trackpad_gesture_common")





    // 1300 - screenshots

    // 1300 - screenshots

    // TODO(b/254513155): Tracking Bug

    @JvmField

    val SCREENSHOT_WORK_PROFILE_POLICY = releasedFlag(1301, "screenshot_work_profile_policy")



    // TODO(b/264916608): Tracking Bug

    // TODO(b/264916608): Tracking Bug

    @JvmField val SCREENSHOT_METADATA = unreleasedFlag(1302, "screenshot_metadata", teamfood = true)

    @JvmField val SCREENSHOT_METADATA = unreleasedFlag(1302, "screenshot_metadata", teamfood = true)

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

+1 −3
Original line number Original line Diff line number Diff line
@@ -39,7 +39,6 @@ import androidx.exifinterface.media.ExifInterface; 




import com.android.internal.annotations.VisibleForTesting;

import com.android.internal.annotations.VisibleForTesting;

import com.android.systemui.flags.FeatureFlags;

import com.android.systemui.flags.FeatureFlags;

import com.android.systemui.flags.Flags;





import com.google.common.util.concurrent.ListenableFuture;

import com.google.common.util.concurrent.ListenableFuture;
@@ -294,8 +293,7 @@ public class ImageExporter { 
            final ContentValues values = createMetadata(time, format, fileName);

            final ContentValues values = createMetadata(time, format, fileName);





            Uri baseUri = MediaStore.Images.Media.EXTERNAL_CONTENT_URI;

            Uri baseUri = MediaStore.Images.Media.EXTERNAL_CONTENT_URI;

            if (flags.isEnabled(Flags.SCREENSHOT_WORK_PROFILE_POLICY)

            if (UserHandle.myUserId() != owner.getIdentifier()) {

                    && UserHandle.myUserId() != owner.getIdentifier()) {

                baseUri = ContentProvider.maybeAddUserId(baseUri, owner.getIdentifier());

                baseUri = ContentProvider.maybeAddUserId(baseUri, owner.getIdentifier());

            }

            }

            Uri uri = resolver.insert(baseUri, values);

            Uri uri = resolver.insert(baseUri, values);

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

+5 −20
Original line number Original line Diff line number Diff line
@@ -47,7 +47,6 @@ import com.android.systemui.R; 
import com.android.systemui.dagger.qualifiers.Background;

import com.android.systemui.dagger.qualifiers.Background;

import com.android.systemui.dagger.qualifiers.Main;

import com.android.systemui.dagger.qualifiers.Main;

import com.android.systemui.flags.FeatureFlags;

import com.android.systemui.flags.FeatureFlags;

import com.android.systemui.flags.Flags;

import com.android.systemui.screenshot.ScrollCaptureController.LongScreenshot;

import com.android.systemui.screenshot.ScrollCaptureController.LongScreenshot;

import com.android.systemui.settings.UserTracker;

import com.android.systemui.settings.UserTracker;
@@ -335,8 +334,7 @@ public class LongScreenshotActivity extends Activity { 
    }

    }





    private void doEdit(Uri uri) {

    private void doEdit(Uri uri) {

        if (mFeatureFlags.isEnabled(Flags.SCREENSHOT_WORK_PROFILE_POLICY) && mScreenshotUserHandle

        if (mScreenshotUserHandle != Process.myUserHandle()) {

                != Process.myUserHandle()) {

            // TODO: Fix transition for work profile. Omitting it in the meantime.

            // TODO: Fix transition for work profile. Omitting it in the meantime.

            mActionExecutor.launchIntentAsync(

            mActionExecutor.launchIntentAsync(

                    ActionIntentCreator.INSTANCE.createEditIntent(uri, this),

                    ActionIntentCreator.INSTANCE.createEditIntent(uri, this),
@@ -365,21 +363,9 @@ public class LongScreenshotActivity extends Activity { 
    }

    }





    private void doShare(Uri uri) {

    private void doShare(Uri uri) {

        if (mFeatureFlags.isEnabled(Flags.SCREENSHOT_WORK_PROFILE_POLICY)) {

        Intent shareIntent = ActionIntentCreator.INSTANCE.createShareIntent(uri);

        Intent shareIntent = ActionIntentCreator.INSTANCE.createShareIntent(uri);

        mActionExecutor.launchIntentAsync(shareIntent, null,

        mActionExecutor.launchIntentAsync(shareIntent, null,

                mScreenshotUserHandle.getIdentifier(), false);

                mScreenshotUserHandle.getIdentifier(), false);

        } else {

            Intent intent = new Intent(Intent.ACTION_SEND);

            intent.setType("image/png");

            intent.putExtra(Intent.EXTRA_STREAM, uri);

            intent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK | Intent.FLAG_ACTIVITY_CLEAR_TASK

                    | Intent.FLAG_GRANT_READ_URI_PERMISSION);

            Intent sharingChooserIntent = Intent.createChooser(intent, null)

                    .addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION);



            startActivityAsUser(sharingChooserIntent, mUserTracker.getUserHandle());

        }

    }

    }





    private void onClicked(View v) {

    private void onClicked(View v) {
@@ -421,8 +407,7 @@ public class LongScreenshotActivity extends Activity { 
        mOutputBitmap = renderBitmap(drawable, bounds);

        mOutputBitmap = renderBitmap(drawable, bounds);

        ListenableFuture<ImageExporter.Result> exportFuture = mImageExporter.export(

        ListenableFuture<ImageExporter.Result> exportFuture = mImageExporter.export(

                mBackgroundExecutor, UUID.randomUUID(), mOutputBitmap, ZonedDateTime.now(),

                mBackgroundExecutor, UUID.randomUUID(), mOutputBitmap, ZonedDateTime.now(),

                mFeatureFlags.isEnabled(Flags.SCREENSHOT_WORK_PROFILE_POLICY)

                mScreenshotUserHandle);

                        ? mScreenshotUserHandle : Process.myUserHandle());

        exportFuture.addListener(() -> onExportCompleted(action, exportFuture), mUiExecutor);

        exportFuture.addListener(() -> onExportCompleted(action, exportFuture), mUiExecutor);

    }

    }

packages/SystemUI/src/com/android/systemui/screenshot/MessageContainerController.kt

+31 −36
Original line number Original line Diff line number Diff line
@@ -47,7 +47,6 @@ constructor( 




    // Minimal implementation for use when Flags.SCREENSHOT_METADATA isn't turned on.

    // Minimal implementation for use when Flags.SCREENSHOT_METADATA isn't turned on.

    fun onScreenshotTaken(userHandle: UserHandle) {

    fun onScreenshotTaken(userHandle: UserHandle) {

        if (featureFlags.isEnabled(Flags.SCREENSHOT_WORK_PROFILE_POLICY)) {

        val workProfileData = workProfileMessageController.onScreenshotTaken(userHandle)

        val workProfileData = workProfileMessageController.onScreenshotTaken(userHandle)

        if (workProfileData != null) {

        if (workProfileData != null) {

            workProfileFirstRunView.visibility = View.VISIBLE

            workProfileFirstRunView.visibility = View.VISIBLE
@@ -61,12 +60,9 @@ constructor( 
            animateInMessageContainer()

            animateInMessageContainer()

        }

        }

    }

    }

    }





    fun onScreenshotTaken(screenshot: ScreenshotData) {

    fun onScreenshotTaken(screenshot: ScreenshotData) {

        if (featureFlags.isEnabled(Flags.SCREENSHOT_WORK_PROFILE_POLICY)) {

        val workProfileData = workProfileMessageController.onScreenshotTaken(screenshot.userHandle)

            val workProfileData =

                workProfileMessageController.onScreenshotTaken(screenshot.userHandle)

        var notifiedApps: List<CharSequence> = listOf()

        var notifiedApps: List<CharSequence> = listOf()

        if (featureFlags.isEnabled(Flags.SCREENSHOT_DETECTION)) {

        if (featureFlags.isEnabled(Flags.SCREENSHOT_DETECTION)) {

            notifiedApps = screenshotDetectionController.maybeNotifyOfScreenshot(screenshot)

            notifiedApps = screenshotDetectionController.maybeNotifyOfScreenshot(screenshot)
@@ -90,7 +86,6 @@ constructor( 
            animateInMessageContainer()

            animateInMessageContainer()

        }

        }

    }

    }

    }





    private fun animateInMessageContainer() {

    private fun animateInMessageContainer() {

        if (container.visibility == View.VISIBLE) return

        if (container.visibility == View.VISIBLE) return

packages/SystemUI/src/com/android/systemui/screenshot/RequestProcessor.kt

+2 −7
Original line number Original line Diff line number Diff line
@@ -23,7 +23,6 @@ import com.android.internal.util.ScreenshotRequest 
import com.android.systemui.dagger.SysUISingleton

import com.android.systemui.dagger.SysUISingleton

import com.android.systemui.dagger.qualifiers.Application

import com.android.systemui.dagger.qualifiers.Application

import com.android.systemui.flags.FeatureFlags

import com.android.systemui.flags.FeatureFlags

import com.android.systemui.flags.Flags.SCREENSHOT_WORK_PROFILE_POLICY

import kotlinx.coroutines.CoroutineScope

import kotlinx.coroutines.CoroutineScope

import kotlinx.coroutines.launch

import kotlinx.coroutines.launch

import java.util.function.Consumer

import java.util.function.Consumer
@@ -57,9 +56,7 @@ class RequestProcessor @Inject constructor( 
        // Whenever displayContentInfo is fetched, the topComponent is also populated

        // Whenever displayContentInfo is fetched, the topComponent is also populated

        // regardless of the managed profile status.

        // regardless of the managed profile status.





        if (request.type != TAKE_SCREENSHOT_PROVIDED_IMAGE &&

        if (request.type != TAKE_SCREENSHOT_PROVIDED_IMAGE) {

                flags.isEnabled(SCREENSHOT_WORK_PROFILE_POLICY)

        ) {





            val info = policy.findPrimaryContent(policy.getDefaultDisplayId())

            val info = policy.findPrimaryContent(policy.getDefaultDisplayId())

            Log.d(TAG, "findPrimaryContent: $info")

            Log.d(TAG, "findPrimaryContent: $info")
@@ -118,9 +115,7 @@ class RequestProcessor @Inject constructor( 
        // Whenever displayContentInfo is fetched, the topComponent is also populated

        // Whenever displayContentInfo is fetched, the topComponent is also populated

        // regardless of the managed profile status.

        // regardless of the managed profile status.





        if (screenshot.type != TAKE_SCREENSHOT_PROVIDED_IMAGE &&

        if (screenshot.type != TAKE_SCREENSHOT_PROVIDED_IMAGE) {

            flags.isEnabled(SCREENSHOT_WORK_PROFILE_POLICY)

        ) {

            val info = policy.findPrimaryContent(policy.getDefaultDisplayId())

            val info = policy.findPrimaryContent(policy.getDefaultDisplayId())

            Log.d(TAG, "findPrimaryContent: $info")

            Log.d(TAG, "findPrimaryContent: $info")

            result.taskId = info.taskId

            result.taskId = info.taskId