Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Crash with QuickSwitch on Android 14 QPR2 #4258

Closed
ammargitham opened this issue Apr 10, 2024 · 7 comments · Fixed by #4417
Closed

[BUG] Crash with QuickSwitch on Android 14 QPR2 #4258

ammargitham opened this issue Apr 10, 2024 · 7 comments · Fixed by #4417
Labels

Comments

@ammargitham
Copy link

ammargitham commented Apr 10, 2024

Describe the bug

The launcher crashes when set as default launcher and also set as recents provider using QuickSwitch

Steps to reproduce

Steps to reproduce the behavior:

  1. Install Launcher
  2. Set as Home app
  3. Set as recent provider
  4. Reboot
  5. See error

Expected behavior

No crash expected

Screenshots

No response

Device information

  • Device: Google Pixel 8 Pro
  • OS: Android 14 QPR2
  • App version: 7516813

Additional context

FATAL EXCEPTION: main
  Process: app.lawnchair.debug, PID: 24269
  java.lang.NoSuchMethodError: No interface method destroyInputConsumer(Ljava/lang/String;I)Z in class Landroid/view/IWindowManager; or its super classes (declaration of 'android.view.IWindowManager' appears in /system/framework/framework.jar!classes4.dex)
  	at com.android.systemui.shared.system.InputConsumerController.registerInputConsumer(InputConsumerController.java:142)
  	at com.android.quickstep.TouchInteractionService.onUserUnlocked(TouchInteractionService.java:567)
  	at com.android.quickstep.TouchInteractionService$$ExternalSyntheticLambda0.run(D8$$SyntheticClass:0)
  	at com.android.launcher3.util.RunnableList.add(RunnableList.java:36)
  	at com.android.launcher3.util.LockedUserState.runOnUserUnlocked(LockedUserState.kt:72)
  	at com.android.quickstep.TouchInteractionService.onCreate(TouchInteractionService.java:504)
  	at android.app.ActivityThread.handleCreateService(ActivityThread.java:4865)
  	at android.app.ActivityThread.-$$Nest$mhandleCreateService(Unknown Source:0)
  	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2365)
  	at android.os.Handler.dispatchMessage(Handler.java:107)
  	at android.os.Looper.loopOnce(Looper.java:232)
  	at android.os.Looper.loop(Looper.java:317)
  	at android.app.ActivityThread.main(ActivityThread.java:8501)
  	at java.lang.reflect.Method.invoke(Native Method)
  	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:552)
  	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:878)

I suspect there has been an api change in recent framework and an updated framework.jar is required. Seems the method now takes an IBinder instead of a String: https://android.googlesource.com/platform/frameworks/base/+/master/core/java/android/view/IWindowManager.aidl#532

@Bennett-69
Copy link

I suspect there has been an api change in recent framework and an updated framework.jar is required. Seems the method now takes an IBinder instead of a String: https://android.googlesource.com/platform/frameworks/base/+/master/core/java/android/view/IWindowManager.aidl#532

Thanks for the extra homework on this. I reported it over a month ago, but it's only just recently been acknowledged as a Lawnchair bug, vs. Quickswitch. Hopefully your details will help expedite the required fix.

#4114 (comment)

@ammargitham
Copy link
Author

#4114 (comment)

Oh, I didn't search for previous ticket for this as I thought no one else may have got QuickSwitch working on QPR2. As it requires some changes with the latest version to work. But yeah. hopefully this gets fixed soon.

@CoelacanthusHex
Copy link

The issue still exists in latest nightly.

@CoelacanthusHex
Copy link

Is #4112 related?

@CoelacanthusHex
Copy link

@MrSluffy, I see you updated framework-14.jar before. Could you update framework-14.jar again?

@CoelacanthusHex
Copy link

CoelacanthusHex commented May 16, 2024

I can confirm it works again with two patches from @nishant6342.
a021440
293d0f4
@nishant6342, are you interested in opening a Pull Request?

@nishant6342
Copy link

I can confirm it works again with two patches from @nishant6342. a021440 293d0f4 @nishant6342, are you interested in opening a Pull Request?

Umm nope, feel free to use it and open pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants