Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Don't override activity display adjustments with app config" into rvc-dev am: ceb2befc1b am: c7a318285d

Browse Source 
Change-Id: Ib84b70f5ecf47c3d5d269ba9d83a73063ab2ecf4
This commit is contained in:
Darryl Johnson
2020-05-01 23:27:14 +00:00
committed by Automerger Merge Worker
parent e7526db875 c7a318285d
commit 9a8414e36b
2 changed files with 73 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
core/java/android/app/ResourcesManager.java
View File

@@ -1109,10 +1109,16 @@ public class ResourcesManager {
Slog.v(TAG, "Changing resources "
+ resourcesImpl + " config to: " + config);
}
int displayId = key.mDisplayId;
final boolean hasOverrideConfiguration = key.hasOverrideConfiguration();
tmpConfig.setTo(config);
// Apply the override configuration before setting the display adjustments to ensure that
// the process config does not override activity display adjustments.
final boolean hasOverrideConfiguration = key.hasOverrideConfiguration();
if (hasOverrideConfiguration) {
tmpConfig.updateFrom(key.mOverrideConfiguration);
}
// Get new DisplayMetrics based on the DisplayAdjustments given to the ResourcesImpl. Update
// a copy if the CompatibilityInfo changed, because the ResourcesImpl object will handle the
// update internally.
@@ -1121,17 +1127,23 @@ public class ResourcesManager {
daj = new DisplayAdjustments(daj);
daj.setCompatibilityInfo(compat);
}
final int displayId = key.mDisplayId;
if (displayId == Display.DEFAULT_DISPLAY) {
daj.setConfiguration(config);
daj.setConfiguration(tmpConfig);
}
DisplayMetrics dm = getDisplayMetrics(displayId, daj);
if (displayId != Display.DEFAULT_DISPLAY) {
applyNonDefaultDisplayMetricsToConfiguration(dm, tmpConfig);
// Re-apply the override configuration to ensure that configuration contexts based on
// a display context (ex: createDisplayContext().createConfigurationContext()) have the
// correct override.
if (hasOverrideConfiguration) {
tmpConfig.updateFrom(key.mOverrideConfiguration);
}
}
if (hasOverrideConfiguration) {
tmpConfig.updateFrom(key.mOverrideConfiguration);
}
resourcesImpl.updateConfiguration(tmpConfig, dm, compat);
}

55
core/tests/coretests/src/android/app/activity/ActivityThreadTest.java
View File

@@ -44,8 +44,11 @@ import android.app.servertransaction.StopActivityItem;
import android.content.Context;
import android.content.Intent;
import android.content.res.Configuration;
import android.content.res.Resources;
import android.graphics.Rect;
import android.hardware.display.DisplayManager;
import android.os.IBinder;
import android.util.DisplayMetrics;
import android.util.MergedConfiguration;
import android.view.Display;
import android.view.View;
@@ -366,6 +369,58 @@ public class ActivityThreadTest {
});
}
@Test
public void testHandleConfigurationChanged_DoesntOverrideActivityConfig() {
final TestActivity activity = mActivityTestRule.launchActivity(new Intent());
InstrumentationRegistry.getInstrumentation().runOnMainSync(() -> {
final Configuration oldActivityConfig =
new Configuration(activity.getResources().getConfiguration());
final DisplayMetrics oldActivityMetrics = new DisplayMetrics();
activity.getDisplay().getMetrics(oldActivityMetrics);
final Resources oldAppResources = activity.getApplication().getResources();
final Configuration oldAppConfig =
new Configuration(oldAppResources.getConfiguration());
final DisplayMetrics oldApplicationMetrics = new DisplayMetrics();
oldApplicationMetrics.setTo(oldAppResources.getDisplayMetrics());
assertEquals("Process config must match the top activity config by default",
0, oldActivityConfig.diffPublicOnly(oldAppConfig));
assertEquals("Process config must match the top activity config by default",
oldActivityMetrics, oldApplicationMetrics);
// Update the application configuration separately from activity config
final Configuration newAppConfig = new Configuration(oldAppConfig);
newAppConfig.densityDpi += 100;
newAppConfig.screenHeightDp += 100;
final Rect newBounds = new Rect(newAppConfig.windowConfiguration.getAppBounds());
newBounds.bottom += 100;
newAppConfig.windowConfiguration.setAppBounds(newBounds);
newAppConfig.windowConfiguration.setBounds(newBounds);
newAppConfig.seq++;
final ActivityThread activityThread = activity.getActivityThread();
activityThread.handleConfigurationChanged(newAppConfig);
// Verify that application config update was applied, but didn't change activity config.
assertEquals("Activity config must not change if the process config changes",
oldActivityConfig, activity.getResources().getConfiguration());
final DisplayMetrics newActivityMetrics = new DisplayMetrics();
activity.getDisplay().getMetrics(newActivityMetrics);
assertEquals("Activity display size must not change if the process config changes",
oldActivityMetrics, newActivityMetrics);
final Resources newAppResources = activity.getApplication().getResources();
assertEquals("Application config must be updated",
newAppConfig, newAppResources.getConfiguration());
final DisplayMetrics newApplicationMetrics = new DisplayMetrics();
newApplicationMetrics.setTo(newAppResources.getDisplayMetrics());
assertNotEquals("Application display size must be updated after config update",
oldApplicationMetrics, newApplicationMetrics);
assertNotEquals("Application display size must be updated after config update",
newActivityMetrics, newApplicationMetrics);
});
}
@Test
public void testResumeAfterNewIntent() {
final Activity activity = mActivityTestRule.launchActivity(new Intent());