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

Consistent crash if video player closed before the video loads #9448

Closed
6 tasks done
Jaydepps opened this issue Nov 23, 2022 · 12 comments · Fixed by #10578
Closed
6 tasks done

Consistent crash if video player closed before the video loads #9448

Jaydepps opened this issue Nov 23, 2022 · 12 comments · Fixed by #10578
Labels
bug Issue is related to a bug player Issues related to any player (main, popup and background)

Comments

@Jaydepps
Copy link

Checklist

  • I am able to reproduce the bug with the latest version.
  • I made sure that there are no existing issues - open or closed - which I could contribute my information to.
  • I have read the FAQ and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.

Affected version

0.24.1

Steps to reproduce the bug

  1. Start playing video
  2. Before video loads back out and close the small player by either taping the x or swiping down

Expected behavior

Video playback stops

Actual behavior

Newpipe crashes

Screenshots/Screen recordings

XRecorder_Edited_23112022_022904.mp4

Logs

Exception

  • User Action: ui error
  • Request: ACRA report
  • Content Country: GB
  • Content Language: en-GB
  • App Language: en_GB
  • Service: none
  • Version: 0.24.1
  • OS: Linux HUAWEI/MAR-LX1AEEA/HWMAR:10/HUAWEIMAR-L21A/10.0.0.233C431:user/release-keys 10 - 29
Crash log

android.app.RemoteServiceException: Context.startForegroundService() did not then call Service.startForeground(): ServiceRecord{e233f40 u0 org.schabi.newpipe/.player.PlayerService}
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2540)
	at android.os.Handler.dispatchMessage(Handler.java:110)
	at android.os.Looper.loop(Looper.java:219)
	at android.app.ActivityThread.main(ActivityThread.java:8668)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:513)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1109)


Affected Android/Custom ROM version

Android 10 / EMUI 12.0.0

Affected device model

Huawei P30 Lite

Additional information

No response

@Jaydepps Jaydepps added bug Issue is related to a bug needs triage Issue is not yet ready for PR authors to take up labels Nov 23, 2022
@opusforlife2
Copy link
Collaborator

What's the value of Smallest width in Developer options?

@opusforlife2 opusforlife2 added the waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed. label Nov 26, 2022
@Jaydepps
Copy link
Author

423 dp (system default)

@Jaydepps Jaydepps reopened this Nov 26, 2022
@github-actions github-actions bot removed the waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed. label Nov 26, 2022
@opusforlife2
Copy link
Collaborator

Your video is blank. Try uploading again?

@opusforlife2 opusforlife2 added the waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed. label Nov 26, 2022
@Jaydepps
Copy link
Author

@github-actions github-actions bot removed the waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed. label Nov 26, 2022
@opusforlife2
Copy link
Collaborator

opusforlife2 commented Nov 26, 2022

Ah, it's the same problem. You'll have to make a new video. Is this playing for you in the browser?

Edit: actually, never mind. I'm pretty sure I know what the bug is, but just to confirm, you try to close the player while the initial buffering is ongoing? Black screen with loading indicator; seekbar is a dark red and no start and end time is shown?

@opusforlife2 opusforlife2 added the waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed. label Nov 26, 2022
@Jaydepps
Copy link
Author

The video is playing fine for me, I checked on multiple devices.

Yes, the crash occurs when you attempt to close the player while its initially loading.

@github-actions github-actions bot removed the waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed. label Nov 26, 2022
@Jaydepps
Copy link
Author

HwVideoEditor_2022_11_26_161133740.mp4

New video

@opusforlife2
Copy link
Collaborator

It's the same issue I reported here: #8646 (comment)

But I only get it on Tablet mode (Smallest width >480 dp). You seem to be getting it even on Phone layout.

Just to cover all bases, is Tablet mode off in Appearance?

@Jaydepps
Copy link
Author

Tablet mode is off, yes.

@opusforlife2 opusforlife2 added player Issues related to any player (main, popup and background) and removed needs triage Issue is not yet ready for PR authors to take up labels Nov 26, 2022
@Douile
Copy link
Contributor

Douile commented Dec 16, 2022

Reproducible on android 11 (lineage 18.1), smallest width 587dp (smallest display and font size in android/display settings), for both tablet mode on and off. Although it doesn't just happen when closing video before load for me: same crash log when notification group is closed while loading, or sometimes while refreshing feed with video playing, or closing playing video while feed is refreshing. Not sure if these are more related to other issue posted but its the same crash log so posted here.

@opusforlife2
Copy link
Collaborator

@Douile Change your smallest width to less than 480 dp and see if it still occurs.

@Douile
Copy link
Contributor

Douile commented Dec 17, 2022

Yes it still occurs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is related to a bug player Issues related to any player (main, popup and background)
Projects
None yet
3 participants