Commit ca62cb72 authored by Andrew Grieve's avatar Andrew Grieve Committed by Commit Bot

Fix Cast-related crash cause by IllegalArgumentException in Android network stack

Bug: b/78588631
Change-Id: Id8ca31426b0a8536dc8d690cc57ff9cd89ee14b7
Reviewed-on: https://chromium-review.googlesource.com/1055610Reviewed-by: default avatarAnthony Berent <aberent@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#559114}
parent 4f2564b3
...@@ -183,7 +183,8 @@ public class MediaUrlResolver extends AsyncTask<Void, Void, MediaUrlResolver.Res ...@@ -183,7 +183,8 @@ public class MediaUrlResolver extends AsyncTask<Void, Void, MediaUrlResolver.Res
Log.e(TAG, "Server response is not valid: %d", responseCode); Log.e(TAG, "Server response is not valid: %d", responseCode);
uri = Uri.EMPTY; uri = Uri.EMPTY;
} }
} catch (IOException e) { } catch (IOException | IllegalArgumentException e) {
// IllegalArgumentException for SSL issue (https://b/78588631).
recordResultHistogram(RESOLVE_RESULT_NETWORK_ERROR); recordResultHistogram(RESOLVE_RESULT_NETWORK_ERROR);
Log.e(TAG, "Failed to fetch the final url", e); Log.e(TAG, "Failed to fetch the final url", e);
uri = Uri.EMPTY; uri = Uri.EMPTY;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment