Merge "Pass DisplayDeviceConfig to AutomaticBrightness" into rvc-dev (b533ef70) · Commits · e / os / android_frameworks_base

services/core/java/com/android/server/display/AutomaticBrightnessController.java

+9 −5

Original line number	Diff line number	Diff line
		@@ -200,7 +200,7 @@ class AutomaticBrightnessController {
		// Context-sensitive brightness configurations require keeping track of the foreground app's
		// package name and category, which is done by registering a TaskStackListener to call back to
		// us onTaskStackChanged, and then using the ActivityTaskManager to get the foreground app's
		// package namd and PackageManager to get its category (so might as well cache them).
		// package name and PackageManager to get its category (so might as well cache them).
		private String mForegroundAppPackageName;
		private String mPendingForegroundAppPackageName;
		private @ApplicationInfo.Category int mForegroundAppCategory;
		@@ -210,6 +210,7 @@ class AutomaticBrightnessController {
		private PackageManager mPackageManager;
		private Context mContext;

		private DisplayDeviceConfig mDisplayDeviceConfig;
		private final Injector mInjector;

		AutomaticBrightnessController(Callbacks callbacks, Looper looper,
		@@ -218,12 +219,14 @@ class AutomaticBrightnessController {
		float dozeScaleFactor, int lightSensorRate, int initialLightSensorRate,
		long brighteningLightDebounceConfig, long darkeningLightDebounceConfig,
		boolean resetAmbientLuxAfterWarmUpConfig, HysteresisLevels ambientBrightnessThresholds,
		HysteresisLevels screenBrightnessThresholds, Context context) {
		HysteresisLevels screenBrightnessThresholds, Context context, DisplayDeviceConfig
		displayDeviceConfig) {
		this(new Injector(), callbacks, looper, sensorManager, lightSensor, mapper,
		lightSensorWarmUpTime, brightnessMin, brightnessMax, dozeScaleFactor,
		lightSensorRate, initialLightSensorRate, brighteningLightDebounceConfig,
		darkeningLightDebounceConfig, resetAmbientLuxAfterWarmUpConfig,
		ambientBrightnessThresholds, screenBrightnessThresholds, context);
		ambientBrightnessThresholds, screenBrightnessThresholds, context,
		displayDeviceConfig);
		}

		@VisibleForTesting
		@@ -233,7 +236,8 @@ class AutomaticBrightnessController {
		float dozeScaleFactor, int lightSensorRate, int initialLightSensorRate,
		long brighteningLightDebounceConfig, long darkeningLightDebounceConfig,
		boolean resetAmbientLuxAfterWarmUpConfig, HysteresisLevels ambientBrightnessThresholds,
		HysteresisLevels screenBrightnessThresholds, Context context) {
		HysteresisLevels screenBrightnessThresholds, Context context, DisplayDeviceConfig
		displayDeviceConfig) {
		mInjector = injector;
		mContext = context;
		mCallbacks = callbacks;
		@@ -260,7 +264,7 @@ class AutomaticBrightnessController {
		mScreenBrightnessThresholds = screenBrightnessThresholds;
		mShortTermModelValid = true;
		mShortTermModelAnchor = -1;

		mDisplayDeviceConfig = displayDeviceConfig;
		mHandler = new AutomaticBrightnessHandler(looper);
		mAmbientLightRingBuffer =
		new AmbientLightRingBuffer(mNormalLightSensorRate, mAmbientLightHorizon);

services/core/java/com/android/server/display/DisplayDevice.java

+11 −0

Original line number	Diff line number	Diff line
		@@ -37,6 +37,7 @@ abstract class DisplayDevice {
		private final DisplayAdapter mDisplayAdapter;
		private final IBinder mDisplayToken;
		private final String mUniqueId;
		private DisplayDeviceConfig mDisplayDeviceConfig;

		// The display device does not manage these properties itself, they are set by
		// the display manager service. The display device shouldn't really be looking at these.
		@@ -68,6 +69,16 @@ abstract class DisplayDevice {
		return mDisplayAdapter;
		}

		/*
		* Gets the DisplayDeviceConfig for this DisplayDevice.
		* Returns null for this device but is overridden in LocalDisplayDevice.
		*
		* @return The DisplayDeviceConfig.
		*/
		public DisplayDeviceConfig getDisplayDeviceConfig() {
		return mDisplayDeviceConfig;
		}

		/**
		* Gets the Surface Flinger display token for this display.
		*

services/core/java/com/android/server/display/DisplayManagerService.java

+2 −1

Original line number	Diff line number	Diff line
		@@ -2476,7 +2476,8 @@ public final class DisplayManagerService extends SystemService {
		}
		};
		mDisplayPowerController = new DisplayPowerController(
		mContext, callbacks, handler, sensorManager, blanker);
		mContext, callbacks, handler, sensorManager, blanker,
		mDisplayDevices.get(Display.DEFAULT_DISPLAY));
		mSensorManager = sensorManager;
		}

services/core/java/com/android/server/display/DisplayPowerController.java

+14 −9

Original line number	Diff line number	Diff line
		@@ -161,6 +161,9 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call
		// The display blanker.
		private final DisplayBlanker mBlanker;

		// The display device.
		private final DisplayDevice mDisplayDevice;

		// Tracker for brightness changes.
		private final BrightnessTracker mBrightnessTracker;

		@@ -348,11 +351,11 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call
		@Nullable
		private BrightnessConfiguration mBrightnessConfiguration;

		// The last brightness that was set by the user and not temporary. Set to -1 when a brightness
		// has yet to be recorded.
		// The last brightness that was set by the user and not temporary. Set to
		// PowerManager.BRIGHTNESS_INVALID_FLOAT when a brightness has yet to be recorded.
		private float mLastUserSetScreenBrightness;

		// The screen brightenss setting has changed but not taken effect yet. If this is different
		// The screen brightness setting has changed but not taken effect yet. If this is different
		// from the current screen brightness setting then this is coming from something other than us
		// and should be considered a user interaction.
		private float mPendingScreenBrightnessSetting;
		@@ -377,8 +380,8 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call
		private float mPendingAutoBrightnessAdjustment;

		// The temporary auto brightness adjustment. Typically set when a user is interacting with the
		// adjustment slider but hasn't settled on a choice yet. Set to Float.NaN when there's no
		// temporary adjustment set.
		// adjustment slider but hasn't settled on a choice yet. Set to
		// PowerManager.BRIGHTNESS_INVALID_FLOAT when there's no temporary adjustment set.
		private float mTemporaryAutoBrightnessAdjustment;

		// Animators.
		@@ -386,27 +389,29 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call
		private ObjectAnimator mColorFadeOffAnimator;
		private RampAnimator<DisplayPowerState> mScreenBrightnessRampAnimator;

		private BrightnessSynchronizer mBrightnessSynchronizer;

		/**
		* Creates the display power controller.
		*/
		public DisplayPowerController(Context context,
		DisplayPowerCallbacks callbacks, Handler handler,
		SensorManager sensorManager, DisplayBlanker blanker) {
		SensorManager sensorManager, DisplayBlanker blanker, DisplayDevice displayDevice) {
		mHandler = new DisplayControllerHandler(handler.getLooper());
		mBrightnessTracker = new BrightnessTracker(context, null);
		mSettingsObserver = new SettingsObserver(mHandler);
		mCallbacks = callbacks;
		mBrightnessSynchronizer = new BrightnessSynchronizer(context);
		mBatteryStats = BatteryStatsService.getService();
		mSensorManager = sensorManager;
		mWindowManagerPolicy = LocalServices.getService(WindowManagerPolicy.class);
		mBlanker = blanker;
		mContext = context;
		mDisplayDevice = displayDevice;

		PowerManager pm = context.getSystemService(PowerManager.class);
		DisplayDeviceConfig displayDeviceConfig = mDisplayDevice.getDisplayDeviceConfig();

		final Resources resources = context.getResources();

		final float screenBrightnessSettingMinimumFloat = clampAbsoluteBrightness(
		pm.getBrightnessConstraint(PowerManager.BRIGHTNESS_CONSTRAINT_TYPE_MINIMUM));

		@@ -498,7 +503,7 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call
		mScreenBrightnessRangeMaximum, dozeScaleFactor, lightSensorRate,
		initialLightSensorRate, brighteningLightDebounce, darkeningLightDebounce,
		autoBrightnessResetAmbientLuxAfterWarmUp, ambientBrightnessThresholds,
		screenBrightnessThresholds, context);
		screenBrightnessThresholds, context, displayDeviceConfig);
		} else {
		mUseSoftwareAutoBrightnessConfig = false;
		}

services/core/java/com/android/server/display/LocalDisplayAdapter.java

+11 −3

Original line number	Diff line number	Diff line
		@@ -203,6 +203,8 @@ final class LocalDisplayAdapter extends DisplayAdapter {
		private Spline mNitsToHalBrightness;
		private boolean mHalBrightnessSupport;

		private DisplayDeviceConfig mDisplayDeviceConfig;

		LocalDisplayDevice(IBinder displayToken, long physicalDisplayId,
		SurfaceControl.DisplayInfo info, SurfaceControl.DisplayConfig[] configs,
		int activeConfigId, SurfaceControl.DesiredDisplayConfigSpecs configSpecs,
		@@ -224,7 +226,7 @@ final class LocalDisplayAdapter extends DisplayAdapter {
		mAllmSupported = SurfaceControl.getAutoLowLatencyModeSupport(displayToken);
		mGameContentTypeSupported = SurfaceControl.getGameContentTypeSupport(displayToken);
		mHalBrightnessSupport = SurfaceControl.getDisplayBrightnessSupport(displayToken);

		mDisplayDeviceConfig = null;
		// Defer configuration file loading
		BackgroundThread.getHandler().sendMessage(PooledLambda.obtainMessage(
		LocalDisplayDevice::loadDisplayConfigurationBrightnessMapping, this));
		@@ -373,17 +375,23 @@ final class LocalDisplayAdapter extends DisplayAdapter {
		return true;
		}

		@Override
		public DisplayDeviceConfig getDisplayDeviceConfig() {
		return mDisplayDeviceConfig;
		}

		private void loadDisplayConfigurationBrightnessMapping() {
		Spline nitsToHal = null;
		Spline sysToNits = null;

		// Load the mapping from nits to HAL brightness range (display-device-config.xml)
		DisplayDeviceConfig config = DisplayDeviceConfig.create(mPhysicalDisplayId);
		mDisplayDeviceConfig = config;
		if (config == null) {
		return;
		}
		final float[] halNits = config.getNits();
		final float[] halBrightness = config.getBrightness();
		final float[] halNits = mDisplayDeviceConfig.getNits();
		final float[] halBrightness = mDisplayDeviceConfig.getBrightness();
		if (halNits == null \|\| halBrightness == null) {
		return;
		}