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

Set last report configuration when attaching process

Browse Source 
The process has received the configuration after bindApplication.
It is unnecessary to send a process configuration change by an
uninitialized last configuration from starting activity
(realStartActivityLocked -> r.setProcess).

Also remove a duplicated invocation of updateConfiguration
because super.onRequestedOverrideConfigurationChanged has
called onConfigurationChanged that also calls updateConfiguration.

Bug: 148604688
Test: atest WindowProcessControllerTests
Test: Enable trace and launch an activity. There should not have
      a configChange between bindApplication and activityStart.
Change-Id: I7afa1278728fbb996f06d74023cc00930eadd3fc
This commit is contained in:
Riddle Hsu
2020-01-13 13:34:09 +08:00
parent 5f87e0dfee
commit 7d5bf33dfc
1 changed files with 8 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
services/core/java/com/android/server/wm/WindowProcessController.java
View File

@@ -178,7 +178,7 @@ public class WindowProcessController extends ConfigurationContainer<Configuratio
private long mLastActivityFinishTime;
// Last configuration that was reported to the process.
private final Configuration mLastReportedConfiguration;
private final Configuration mLastReportedConfiguration = new Configuration();
// Configuration that is waiting to be dispatched to the process.
private Configuration mPendingConfiguration;
private final Configuration mNewOverrideConfig = new Configuration();
@@ -192,7 +192,7 @@ public class WindowProcessController extends ConfigurationContainer<Configuratio
/** Whether our process is currently running a {@link IRemoteAnimationRunner} */
private boolean mRunningRemoteAnimation;
public WindowProcessController(ActivityTaskManagerService atm, ApplicationInfo info,
public WindowProcessController(@NonNull ActivityTaskManagerService atm, ApplicationInfo info,
String name, int uid, int userId, Object owner, WindowProcessListener listener) {
mInfo = info;
mName = name;
@@ -201,11 +201,8 @@ public class WindowProcessController extends ConfigurationContainer<Configuratio
mOwner = owner;
mListener = listener;
mAtm = atm;
mLastReportedConfiguration = new Configuration();
mDisplayId = INVALID_DISPLAY;
if (atm != null) {
onConfigurationChanged(atm.getGlobalConfiguration());
}
onConfigurationChanged(atm.getGlobalConfiguration());
}
public void setPid(int pid) {
@@ -220,6 +217,11 @@ public class WindowProcessController extends ConfigurationContainer<Configuratio
public void setThread(IApplicationThread thread) {
synchronized (mAtm.mGlobalLockWithoutBoost) {
mThread = thread;
// In general this is called from attaching application, so the last configuration
// has been sent to client by {@link android.app.IApplicationThread#bindApplication}.
// If this process is system server, it is fine because system is booting and a new
// configuration will update when display is ready.
setLastReportedConfiguration(getConfiguration());
}
}
@@ -1060,7 +1062,6 @@ public class WindowProcessController extends ConfigurationContainer<Configuratio
mNewOverrideConfig.setTo(mergedOverrideConfig);
mNewOverrideConfig.windowConfiguration.setActivityType(ACTIVITY_TYPE_UNDEFINED);
super.onRequestedOverrideConfigurationChanged(mNewOverrideConfig);
updateConfiguration();
}
private void updateConfiguration() {