Skip to content

Commit f18ee8e

Browse files
committed
Added a bit more documentation
1 parent fb58967 commit f18ee8e

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

app/src/main/java/org/schabi/newpipe/fragments/detail/VideoDetailPlayerCrasher.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,14 @@
2727
import java.util.concurrent.TimeoutException;
2828
import java.util.function.Supplier;
2929

30+
/**
31+
* Outsourced logic for crashing the player in the {@link VideoDetailFragment}.
32+
*/
3033
public class VideoDetailPlayerCrasher {
3134

35+
// This has to be <= 23 chars on devices running Android 7 or lower (API <= 25)
36+
// or it fails with an IllegalArgumentException
37+
// https://stackoverflow.com/a/54744028
3238
private static final String TAG = "VideoDetPlayerCrasher";
3339

3440
@NonNull
@@ -109,6 +115,8 @@ public void onCrashThePlayer(final Player player) {
109115
return;
110116
}
111117

118+
// -- Build the dialog/UI --
119+
112120
final Context themeWrapperContext = getThemeWrapperContext();
113121

114122
final LayoutInflater inflater = LayoutInflater.from(themeWrapperContext);

0 commit comments

Comments
 (0)