Set FLAG_HARDWARE_ACCELERATED in ShadowPackageManger.addActivityIfNotPresent

integration_tests/nativegraphics/src/test/java/org/robolectric/integrationtests/nativegraphics/HardwareAcceleratedActivityRenderTest.java

@@ -13,7 +13,6 @@
 import android.view.View;
 import android.view.ViewGroup.LayoutParams;
 import android.view.Window;
-import android.view.WindowManager;
 import android.widget.FrameLayout;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
@@ -57,12 +56,6 @@ static class HardwareAcceleratedActivity extends Activity {
     @Override
     public void onCreate(Bundle savedInstanceState) {
       super.onCreate(savedInstanceState);
-      // TODO(hoisie): manually setting these flags should not be required. Robolectric should
-      // set them automatically by default (they have been default since ICS).
-      getWindow()
-          .setFlags(
-              WindowManager.LayoutParams.FLAG_HARDWARE_ACCELERATED,
-              WindowManager.LayoutParams.FLAG_HARDWARE_ACCELERATED);
       FrameLayout frameLayout = new FrameLayout(this);
       frameLayout.setLayoutParams(
           new LayoutParams(LayoutParams.MATCH_PARENT, LayoutParams.MATCH_PARENT));

shadows/framework/src/main/java/org/robolectric/shadows/HardwareRenderingScreenshot.java

@@ -42,10 +42,11 @@ private HardwareRenderingScreenshot() {}
    * the presence of the {@link #USE_HARDWARE_RENDERER_NATIVE_ENV} property, and the {@link
    * GraphicsMode}.
    */
-  static boolean canTakeScreenshot() {
+  static boolean canTakeScreenshot(View view) {
     return VERSION.SDK_INT >= VERSION_CODES.S
         && "hardware".equalsIgnoreCase(System.getProperty(PIXEL_COPY_RENDER_MODE, ""))
-        && ShadowView.useRealGraphics();
+        && ShadowView.useRealGraphics()
+        && view.canHaveDisplayList();
   }
 
   /**

shadows/framework/src/main/java/org/robolectric/shadows/ShadowPackageManager.java

@@ -204,12 +204,10 @@ public class ShadowPackageManager {
    * @return existing or newly created activity info.
    */
   public ActivityInfo addActivityIfNotPresent(ComponentName componentName) {
+    ActivityInfo activityInfo = updateName(componentName, new ActivityInfo());
+    activityInfo.flags |= ActivityInfo.FLAG_HARDWARE_ACCELERATED;
     return addComponent(
-        activityFilters,
-        p -> p.activities,
-        (p, a) -> p.activities = a,
-        updateName(componentName, new ActivityInfo()),
-        false);
+        activityFilters, p -> p.activities, (p, a) -> p.activities = a, activityInfo, false);
   }
 
   /**

shadows/framework/src/main/java/org/robolectric/shadows/ShadowPixelCopy.java

@@ -168,7 +168,7 @@ private static void takeScreenshot(View view, Bitmap screenshot, @Nullable Rect
 
     Bitmap bitmap = Bitmap.createBitmap(view.getWidth(), view.getHeight(), Bitmap.Config.ARGB_8888);
 
-    if (HardwareRenderingScreenshot.canTakeScreenshot()) {
+    if (HardwareRenderingScreenshot.canTakeScreenshot(view)) {
       PerfStatsCollector.getInstance()
           .measure(
               "ShadowPixelCopy-Hardware",

shadows/framework/src/main/java/org/robolectric/shadows/ShadowUiAutomation.java

@@ -131,7 +131,7 @@ protected Bitmap takeScreenshot() throws Exception {
                 Bitmap window =
                     Bitmap.createBitmap(
                         rootView.getWidth(), rootView.getHeight(), Bitmap.Config.ARGB_8888);
-                if (HardwareRenderingScreenshot.canTakeScreenshot()) {
+                if (HardwareRenderingScreenshot.canTakeScreenshot(rootView)) {
                   HardwareRenderingScreenshot.takeScreenshot(rootView, window);
                 } else {
                   Canvas windowCanvas = new Canvas(window);