loliball/media3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

In DemoUtil, don't set cookie handler when using HttpEngineDataSource.

Browse Source 
HttpEngine does not support cookie storage.

#cherrypick

PiperOrigin-RevId: 646084702
This commit is contained in:
okunhardt 2024-06-24 07:18:11 -07:00 committed by Copybara-Service
parent 938fac4161
commit bb568b5150
1 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
demos/main/src/main/java/androidx/media3/demo/main/DemoUtil.java
View File

@ -106,7 +106,6 @@ public final class DemoUtil {
}
context = context.getApplicationContext();
if (Build.VERSION.SDK_INT >= 34) {
setCookieHandler();
HttpEngine httpEngine = new HttpEngine.Builder(context).build();
httpDataSourceFactory =
new HttpEngineDataSource.Factory(httpEngine, Executors.newSingleThreadExecutor());
@ -122,15 +121,11 @@ public final class DemoUtil {
}
// The device doesn't support HttpEngine or we don't want to allow Cronet, or we failed to
// instantiate a CronetEngine.
setCookieHandler();
httpDataSourceFactory = new DefaultHttpDataSource.Factory();
return httpDataSourceFactory;
}
private static void setCookieHandler() {
CookieManager cookieManager = new CookieManager();
cookieManager.setCookiePolicy(CookiePolicy.ACCEPT_ORIGINAL_SERVER);
CookieHandler.setDefault(cookieManager);
httpDataSourceFactory = new DefaultHttpDataSource.Factory();
return httpDataSourceFactory;
}
/** Returns a {@link DataSource.Factory}. */