New issue
Advanced search Search tips

Issue 719394 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Deal with java exceptions thrown in media::MediaCodecUtil::IsKnownUnaccelerated

Project Member Reported by gsennton@chromium.org, May 8 2017

Issue description

There is already a bug open for this call stack: crbug.com/674314 where Torne found some exceptions that might have caused this (by querying for Chrome crashes with similar call stacks).

Given that we should see these exceptions in Chrome anyway, should we just leave this one be?

AFAICT this thread is not backed by Java either.
 
Blocking: 592556
Status: Available (was: Untriaged)
Moving to "available" as per new bug triaging guidelines
Thanks Nate!
Project Member

Comment 4 by sheriffbot@chromium.org, May 9 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
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

Comment 5 by boliu@chromium.org, May 9 2018

Status: WontFix (was: Untriaged)
this is from our own code, and should crash

Comment 6 by torne@chromium.org, May 9 2018

Status: Available (was: WontFix)
Summary: Deal with java exceptions thrown in media::MediaCodecUtil::IsKnownUnaccelerated (was: [JNI crashes] Properly throw Java Exceptions from media::MediaCodecUtil::IsKnownUnaccelerated)
I don't think this should crash - we should probably just catch and handle/ignore the exception in Java instead of letting it propagate to native in the first place. This is one of those areas where vendor code may throw undocumented exceptions and we just have to deal with it.

Comment 7 by torne@chromium.org, May 9 2018

Blocking: -592556

Comment 8 by boliu@chromium.org, May 9 2018

which is tracked in crbug.com/674314?

Comment 9 by torne@chromium.org, May 9 2018

Status: WontFix (was: Available)
Yeah. Thanks for spotting that Bo!

Sign in to add a comment