Merge changes Ibe0e4326,I0b50d678 (5190320c) · Commits · e / os / platform_frameworks_native

services/inputflinger/dispatcher/InputDispatcher.cpp

+7 −0

Original line number	Diff line number	Diff line
		@@ -6371,6 +6371,13 @@ void InputDispatcher::onWindowInfosChanged(const std::vector<WindowInfo>& window

		{ // acquire lock
		std::scoped_lock _l(mLock);

		// Ensure that we have an entry created for all existing displays so that if a displayId has
		// no windows, we can tell that the windows were removed from the display.
		for (const auto& [displayId, _] : mWindowHandlesByDisplay) {
		handlesPerDisplay[displayId];
		}

		mDisplayInfos.clear();
		for (const auto& displayInfo : displayInfos) {
		mDisplayInfos.emplace(displayInfo.displayId, displayInfo);

services/inputflinger/tests/InputDispatcher_test.cpp

+29 −0

Original line number	Diff line number	Diff line
		@@ -2370,6 +2370,35 @@ TEST_F(InputDispatcherTest, ActionOutsideSentOnlyWhenAWindowIsTouched) {
		thirdWindow->consumeMotionDown();
		}

		TEST_F(InputDispatcherTest, OnWindowInfosChanged_RemoveAllWindowsOnDisplay) {
		std::shared_ptr<FakeApplicationHandle> application = std::make_shared<FakeApplicationHandle>();
		sp<FakeWindowHandle> window =
		new FakeWindowHandle(application, mDispatcher, "Fake Window", ADISPLAY_ID_DEFAULT);
		window->setFocusable(true);

		mDispatcher->onWindowInfosChanged({*window->getInfo()}, {});
		setFocusedWindow(window);

		window->consumeFocusEvent(true);

		NotifyKeyArgs keyDown = generateKeyArgs(AKEY_EVENT_ACTION_DOWN, ADISPLAY_ID_DEFAULT);
		NotifyKeyArgs keyUp = generateKeyArgs(AKEY_EVENT_ACTION_UP, ADISPLAY_ID_DEFAULT);
		mDispatcher->notifyKey(&keyDown);
		mDispatcher->notifyKey(&keyUp);

		window->consumeKeyDown(ADISPLAY_ID_DEFAULT);
		window->consumeKeyUp(ADISPLAY_ID_DEFAULT);

		// All windows are removed from the display. Ensure that we can no longer dispatch to it.
		mDispatcher->onWindowInfosChanged({}, {});

		window->consumeFocusEvent(false);

		mDispatcher->notifyKey(&keyDown);
		mDispatcher->notifyKey(&keyUp);
		window->assertNoEvents();
		}

		/**
		* Ensure the correct coordinate spaces are used by InputDispatcher.
		*

services/surfaceflinger/Layer.cpp

+14 −3

Original line number	Diff line number	Diff line
		@@ -78,6 +78,7 @@
		namespace android {
		namespace {
		constexpr int kDumpTableRowLength = 159;
		const ui::Transform kIdentityTransform;
		} // namespace

		using namespace ftl::flag_operators;
		@@ -2209,7 +2210,8 @@ void Layer::writeToProtoCommonState(LayerProto* layerInfo, LayerVector::StateSet
		if ((traceFlags & LayerTracing::TRACE_INPUT) && needsInputInfo()) {
		WindowInfo info;
		if (useDrawing) {
		info = fillInputInfo(ui::Transform(), /* displayIsSecure */ true);
		info = fillInputInfo(
		InputDisplayArgs{.transform = &kIdentityTransform, .isSecure = true});
		} else {
		info = state.inputInfo;
		}
		@@ -2416,7 +2418,7 @@ void Layer::handleDropInputMode(gui::WindowInfo& info) const {
		}
		}

		WindowInfo Layer::fillInputInfo(const ui::Transform& displayTransform, bool displayIsSecure) {
		WindowInfo Layer::fillInputInfo(const InputDisplayArgs& displayArgs) {
		if (!hasInputInfo()) {
		mDrawingState.inputInfo.name = getName();
		mDrawingState.inputInfo.ownerUid = mOwnerUid;
		@@ -2425,12 +2427,21 @@ WindowInfo Layer::fillInputInfo(const ui::Transform& displayTransform, bool disp
		mDrawingState.inputInfo.displayId = getLayerStack().id;
		}

		const ui::Transform& displayTransform =
		displayArgs.transform != nullptr ? *displayArgs.transform : kIdentityTransform;

		WindowInfo info = mDrawingState.inputInfo;
		info.id = sequence;
		info.displayId = getLayerStack().id;

		fillInputFrameInfo(info, displayTransform);

		if (displayArgs.transform == nullptr) {
		// Do not let the window receive touches if it is not associated with a valid display
		// transform. We still allow the window to receive keys and prevent ANRs.
		info.inputConfig \|= WindowInfo::InputConfig::NOT_TOUCHABLE;
		}

		// For compatibility reasons we let layers which can receive input
		// receive input before they have actually submitted a buffer. Because
		// of this we use canReceiveInput instead of isVisible to check the
		@@ -2448,7 +2459,7 @@ WindowInfo Layer::fillInputInfo(const ui::Transform& displayTransform, bool disp

		// If the window will be blacked out on a display because the display does not have the secure
		// flag and the layer has the secure flag set, then drop input.
		if (!displayIsSecure && isSecure()) {
		if (!displayArgs.isSecure && isSecure()) {
		info.inputConfig \|= WindowInfo::InputConfig::DROP_INPUT;
		}

services/surfaceflinger/Layer.h

+5 −1

Original line number	Diff line number	Diff line
		@@ -854,7 +854,11 @@ public:
		bool getPremultipledAlpha() const;
		void setInputInfo(const gui::WindowInfo& info);

		gui::WindowInfo fillInputInfo(const ui::Transform& displayTransform, bool displayIsSecure);
		struct InputDisplayArgs {
		const ui::Transform* transform = nullptr;
		bool isSecure = false;
		};
		gui::WindowInfo fillInputInfo(const InputDisplayArgs& displayArgs);

		/**
		* Returns whether this layer has an explicitly set input-info.

services/surfaceflinger/SurfaceFlinger.cpp

+5 −5

Original line number	Diff line number	Diff line
		@@ -3260,11 +3260,11 @@ void SurfaceFlinger::buildWindowInfos(std::vector<WindowInfo>& outWindowInfos,
		mDrawingState.traverseInReverseZOrder([&](Layer* layer) {
		if (!layer->needsInputInfo()) return;

		// Do not create WindowInfos for windows on displays that cannot receive input.
		if (const auto opt = displayInputInfos.get(layer->getLayerStack())) {
		const auto& info = opt->get();
		outWindowInfos.push_back(layer->fillInputInfo(info.transform, info.isSecure));
		}
		const auto opt = displayInputInfos.get(layer->getLayerStack(),
		[](const auto& info) -> Layer::InputDisplayArgs {
		return {&info.transform, info.isSecure};
		});
		outWindowInfos.push_back(layer->fillInputInfo(opt.value_or(Layer::InputDisplayArgs{})));
		});

		sNumWindowInfos = outWindowInfos.size();