Snap for 8456972 from 4809267e7b873d1c12b45a975163a3d934d1a0f3 to tm-release (bf442939) · Commits · e / os / android_frameworks_native

data/etc/Android.bp

+12 −0

Original line number	Diff line number	Diff line
		@@ -226,6 +226,12 @@ prebuilt_etc {
		defaults: ["frameworks_native_data_etc_defaults"],
		}

		prebuilt_etc {
		name: "android.software.opengles.deqp.level-2022-03-01.prebuilt.xml",
		src: "android.software.opengles.deqp.level-2022-03-01.xml",
		defaults: ["frameworks_native_data_etc_defaults"],
		}

		prebuilt_etc {
		name: "android.software.sip.voip.prebuilt.xml",
		src: "android.software.sip.voip.xml",
		@@ -244,6 +250,12 @@ prebuilt_etc {
		defaults: ["frameworks_native_data_etc_defaults"],
		}

		prebuilt_etc {
		name: "android.software.vulkan.deqp.level-2022-03-01.prebuilt.xml",
		src: "android.software.vulkan.deqp.level-2022-03-01.xml",
		defaults: ["frameworks_native_data_etc_defaults"],
		}

		prebuilt_etc {
		name: "aosp_excluded_hardware.prebuilt.xml",
		src: "aosp_excluded_hardware.xml",

services/inputflinger/reader/mapper/CursorInputMapper.cpp

+18 −6

Original line number	Diff line number	Diff line
		@@ -27,6 +27,9 @@

		namespace android {

		// The default velocity control parameters that has no effect.
		static const VelocityControlParameters FLAT_VELOCITY_CONTROL_PARAMS{};

		// --- CursorMotionAccumulator ---

		CursorMotionAccumulator::CursorMotionAccumulator() {
		@@ -154,8 +157,9 @@ void CursorInputMapper::configure(nsecs_t when, const InputReaderConfiguration*
		mHWheelScale = 1.0f;
		}

		if ((!changes && config->pointerCaptureRequest.enable) \|\|
		(changes & InputReaderConfiguration::CHANGE_POINTER_CAPTURE)) {
		const bool configurePointerCapture = (!changes && config->pointerCaptureRequest.enable) \|\|
		(changes & InputReaderConfiguration::CHANGE_POINTER_CAPTURE);
		if (configurePointerCapture) {
		if (config->pointerCaptureRequest.enable) {
		if (mParameters.mode == Parameters::MODE_POINTER) {
		mParameters.mode = Parameters::MODE_POINTER_RELATIVE;
		@@ -180,11 +184,19 @@ void CursorInputMapper::configure(nsecs_t when, const InputReaderConfiguration*
		}
		}

		if (!changes \|\| (changes & InputReaderConfiguration::CHANGE_POINTER_SPEED)) {
		if (!changes \|\| (changes & InputReaderConfiguration::CHANGE_POINTER_SPEED) \|\|
		configurePointerCapture) {
		if (config->pointerCaptureRequest.enable) {
		// Disable any acceleration or scaling when Pointer Capture is enabled.
		mPointerVelocityControl.setParameters(FLAT_VELOCITY_CONTROL_PARAMS);
		mWheelXVelocityControl.setParameters(FLAT_VELOCITY_CONTROL_PARAMS);
		mWheelYVelocityControl.setParameters(FLAT_VELOCITY_CONTROL_PARAMS);
		} else {
		mPointerVelocityControl.setParameters(config->pointerVelocityControlParameters);
		mWheelXVelocityControl.setParameters(config->wheelVelocityControlParameters);
		mWheelYVelocityControl.setParameters(config->wheelVelocityControlParameters);
		}
		}

		if (!changes \|\| (changes & InputReaderConfiguration::CHANGE_DISPLAY_INFO)) {
		mOrientation = DISPLAY_ORIENTATION_0;

services/inputflinger/tests/InputReader_test.cpp

+57 −1

Original line number	Diff line number	Diff line
		@@ -375,6 +375,11 @@ public:

		float getPointerGestureMovementSpeedRatio() { return mConfig.pointerGestureMovementSpeedRatio; }

		void setVelocityControlParams(const VelocityControlParameters& params) {
		mConfig.pointerVelocityControlParameters = params;
		mConfig.wheelVelocityControlParameters = params;
		}

		private:
		uint32_t mNextPointerCaptureSequenceNumber = 0;

		@@ -2949,7 +2954,10 @@ protected:
		}

		void configureDevice(uint32_t changes) {
		if (!changes \|\| (changes & InputReaderConfiguration::CHANGE_DISPLAY_INFO)) {
		if (!changes \|\|
		(changes &
		(InputReaderConfiguration::CHANGE_DISPLAY_INFO \|
		InputReaderConfiguration::CHANGE_POINTER_CAPTURE))) {
		mReader->requestRefreshConfiguration(changes);
		mReader->loopOnce();
		}
		@@ -4842,6 +4850,54 @@ TEST_F(CursorInputMapperTest, Process_PointerCapture) {
		ASSERT_NO_FATAL_FAILURE(assertPosition(*mFakePointerController, 110.0f, 220.0f));
		}

		/**
		* When Pointer Capture is enabled, we expect to report unprocessed relative movements, so any
		* pointer acceleration or speed processing should not be applied.
		*/
		TEST_F(CursorInputMapperTest, PointerCaptureDisablesVelocityProcessing) {
		addConfigurationProperty("cursor.mode", "pointer");
		const VelocityControlParameters testParams(5.f /scale/, 0.f /low threshold/,
		100.f /high threshold/, 10.f /acceleration/);
		mFakePolicy->setVelocityControlParams(testParams);
		CursorInputMapper& mapper = addMapperAndConfigure<CursorInputMapper>();

		NotifyDeviceResetArgs resetArgs;
		ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyDeviceResetWasCalled(&resetArgs));
		ASSERT_EQ(ARBITRARY_TIME, resetArgs.eventTime);
		ASSERT_EQ(DEVICE_ID, resetArgs.deviceId);

		NotifyMotionArgs args;

		// Move and verify scale is applied.
		process(mapper, ARBITRARY_TIME, READ_TIME, EV_REL, REL_X, 10);
		process(mapper, ARBITRARY_TIME, READ_TIME, EV_REL, REL_Y, 20);
		process(mapper, ARBITRARY_TIME, READ_TIME, EV_SYN, SYN_REPORT, 0);
		ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasCalled(&args));
		ASSERT_EQ(AINPUT_SOURCE_MOUSE, args.source);
		ASSERT_EQ(AMOTION_EVENT_ACTION_HOVER_MOVE, args.action);
		const float relX = args.pointerCoords[0].getAxisValue(AMOTION_EVENT_AXIS_RELATIVE_X);
		const float relY = args.pointerCoords[0].getAxisValue(AMOTION_EVENT_AXIS_RELATIVE_Y);
		ASSERT_GT(relX, 10);
		ASSERT_GT(relY, 20);

		// Enable Pointer Capture
		mFakePolicy->setPointerCapture(true);
		configureDevice(InputReaderConfiguration::CHANGE_POINTER_CAPTURE);
		NotifyPointerCaptureChangedArgs captureArgs;
		ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyCaptureWasCalled(&captureArgs));
		ASSERT_TRUE(captureArgs.request.enable);

		// Move and verify scale is not applied.
		process(mapper, ARBITRARY_TIME, READ_TIME, EV_REL, REL_X, 10);
		process(mapper, ARBITRARY_TIME, READ_TIME, EV_REL, REL_Y, 20);
		process(mapper, ARBITRARY_TIME, READ_TIME, EV_SYN, SYN_REPORT, 0);
		ASSERT_NO_FATAL_FAILURE(mFakeListener->assertNotifyMotionWasCalled(&args));
		ASSERT_EQ(AINPUT_SOURCE_MOUSE_RELATIVE, args.source);
		ASSERT_EQ(AMOTION_EVENT_ACTION_MOVE, args.action);
		ASSERT_EQ(10, args.pointerCoords[0].getX());
		ASSERT_EQ(20, args.pointerCoords[0].getY());
		}

		TEST_F(CursorInputMapperTest, Process_ShouldHandleDisplayId) {
		CursorInputMapper& mapper = addMapperAndConfigure<CursorInputMapper>();