[JNI crashes] Properly throw Java Exceptions from android_webview::InputStreamImpl::Read |
||||||
Issue descriptionandroid_webview::InputStreamImpl::Read calls into Java through Java_InputStreamUtil_read, which call the read() method on a InputStream handed to us by the embedding app. Crash.corp url: https://crash.corp.google.com/browse?q=custom_data.ChromeCrashProto.magic_signature_1.name%20CONTAINS%20%27%5BAndroid%20Java%20Exception%5D%27%20AND%20product.name%3D%27AndroidWebView%27%20AND%20NOT%20product.Version%20CONTAINS%20%2754.0.2%27%20AND%20NOT%20product.Version%20CONTAINS%20%2753.0.2%27%20AND%20NOT%20product.Version%20CONTAINS%20%2752.0.2%27%20AND%20NOT%20product.Version%20CONTAINS%20%2751.0.2%27%20AND%20special_protos.user_feedback.mobile_data.crash_data.exception_class_name%3D%27Native%20crash%27%20AND%20custom_data.ChromeCrashProto.magic_signature_1.name%3D%27%5BAndroid%20Java%20Exception%5D%20android_webview%3A%3AInputStreamImpl%3A%3ARead%27&ignore_case=false&enable_rewrite=true&omit_field_name=&omit_field_value=&omit_field_opt=%3D
,
May 5 2017
yep
,
May 8 2017
,
May 8 2017
The thread that is used is one in a thread pool: https://cs.chromium.org/chromium/src/android_webview/browser/net/android_stream_reader_url_request_job.cc?l=220&rcl=adf8886b489461915a26a3e2d70d106e63c232d2 though it looks like using that thread pool (or at least the task-runner) is deprecated: https://cs.chromium.org/chromium/src/content/public/browser/browser_thread.h?l=226&rcl=8e987fe4730ac5991bb5091c449f7e0db81b60ea
,
May 9 2017
,
May 9 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 9 2018
I guess this depends on the "java TaskScheduler" idea being thrown around in crbug.com/825947
,
May 9 2018
Or on just making more/all threads into java threads as discussed for shouldInterceptRequest.
,
Jun 6 2018
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by gsennton@chromium.org
, May 5 2017