BufferedInputStream cannot be cast to class AssetManager$AssetInputStream [226457466]

Fixed

Bug

Status Update

No update yet.

Description

tn...@google.com

created issue #1

Mar 23, 2022 11:34PM

My custom view hits a cast exception when running in layoutlib (Tiramisu preview).

java.lang.ClassCastException: class java.io.BufferedInputStream cannot be cast to class android.content.res.AssetManager$AssetInputStream (java.io.BufferedInputStream is in module java.base of loader 'bootstrap'; android.content.res.AssetManager$AssetInputStream is in unnamed module of loader com.intellij.ide.plugins.cl.PluginClassLoader @7ad8bb39)
	at android.graphics.ImageDecoder$ResourceSource.createImageDecoder(ImageDecoder.java:486)
	at android.graphics.ImageDecoder.decodeBitmapImpl_Original(ImageDecoder.java:1870)
	at android.graphics.ImageDecoder_Delegate.decodeBitmapImpl(ImageDecoder_Delegate.java:42)
	at android.graphics.ImageDecoder.decodeBitmapImpl(ImageDecoder.java:1870)
	at android.graphics.ImageDecoder.decodeBitmap(ImageDecoder.java:1969)
	at com.android.tools.test.shaderexperiment.BitmapTransitionView.<init>(BitmapTransitionView.kt:18)
	at com.android.tools.test.shaderexperiment.BitmapTransitionView.<init>(BitmapTransitionView.kt:11)
	at com.android.tools.test.shaderexperiment.BitmapTransitionView.<init>(BitmapTransitionView.kt)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:490)
	at org.jetbrains.android.uipreview.ViewLoader.createNewInstance(ViewLoader.java:325)
	at org.jetbrains.android.uipreview.ViewLoader.loadClass(ViewLoader.java:174)
	at org.jetbrains.android.uipreview.ViewLoader.loadView(ViewLoader.java:135)
	at com.android.tools.idea.rendering.LayoutlibCallbackImpl.loadView(LayoutlibCallbackImpl.java:294)
	at android.view.BridgeInflater.loadCustomView(BridgeInflater.java:417)
	at android.view.BridgeInflater.loadCustomView(BridgeInflater.java:428)
	at android.view.BridgeInflater.createViewFromTag(BridgeInflater.java:332)
	at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:965)
	at android.view.LayoutInflater.inflate(LayoutInflater.java:663)
	at android.view.LayoutInflater.inflate(LayoutInflater.java:505)
	at com.android.layoutlib.bridge.impl.RenderSessionImpl.inflate(RenderSessionImpl.java:360)
	at com.android.layoutlib.bridge.Bridge.createSession(Bridge.java:436)
	at com.android.tools.idea.layoutlib.LayoutLibrary.createSession(LayoutLibrary.java:121)
	at com.android.tools.idea.rendering.RenderTask.createRenderSession(RenderTask.java:719)
	at com.android.tools.idea.rendering.RenderTask.lambda$inflate$9(RenderTask.java:875)
	at com.android.tools.idea.rendering.RenderExecutor$runAsyncActionWithTimeout$2.run(RenderExecutor.kt:187)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:829)

The purpose of this was to test AGSL in layoutlib but I think I'm hitting an earlier problem related to loading assets from a custom view in layoutlib, using this call:

ImageDecoder.decodeBitmap(
        ImageDecoder.createSource(context.resources, R.drawable.large_photo))

ShaderExperiment.zip

2.4 MB

Download

Comments

uc...@google.com <uc...@google.com> Mar 28, 2022 05:23AM

Reassigned to an...@google.com.

jg...@google.com <jg...@google.com> Mar 28, 2022 10:31AM

Reassigned to jg...@google.com.

ku...@google.com <ku...@google.com> #2Oct 13, 2022 09:06PM

Marked as fixed, reassigned to ku...@google.com.

I don't think the average developer understands the distinction between method references and method definitions in this context (inside a DEX file). I know I don't.

@jvg, can you elaborate?

Screen Shot 2022-10-13 at 22.05.41.png

1007 KB

View

Download

an...@google.com <an...@google.com> #3Sep 6, 2023 06:23PM

people who are looking at the bytecode in the dex to optimize their app through shrinking hopefully will learn the difference as string tables for method references between various dex files can account for quite a bit of disk space. but more importantly a definition is something you can analyze its code for where a reference you can't so the UI can be confusing without seeing the difference

an...@google.com <an...@google.com> #4Sep 12, 2024 06:57PM

How about something like this?

I think having defined members be directly under the class node while references are nested in a group makes more sense.

Also note that the method refs have a slightly different icon. There is no existing equivalent for field refs that I could find.

https://screenshot.googleplex.com/4yXVM9Rzh4wnkjM.png

Issue 226457466