Commit 5964dbd4 authored by Trevor Perrier's avatar Trevor Perrier Committed by Commit Bot

[Android] Log ResourceBundle path status

This CL adds logging to help determine where an issue loading native
resources with an override locale present occurs.

One theory is that checks in ResourceBundle.getLocalePakResourcePath
fail because the Java context can not find override assets.  This CL
logs whether or not the asset path was found.  If it is found we will
know that the issue is on the native side, if it isn't found we know
the (current) blocker is on the Java side.

Bug: 1143026
Change-Id: Iab1275cd5b0dd09e12808bafa4379ac3f519b126
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2520421Reviewed-by: default avatarAndrew Grieve <agrieve@chromium.org>
Reviewed-by: default avatarTheresa  <twellington@chromium.org>
Commit-Queue: Trevor  Perrier <perrier@chromium.org>
Cr-Commit-Position: refs/heads/master@{#825092}
parent 82c8b6f3
......@@ -103,8 +103,10 @@ public final class ResourceBundle {
// The file may not exist if the language split for this locale has not been installed
// yet, so make sure it exists before returning the asset path.
try (AssetFileDescriptor afd = manager.openNonAssetFd(assetPath)) {
Log.i(TAG, "Found: %s", assetPath);
return assetPath;
} catch (IOException e) {
Log.e(TAG, "Not found: %s", assetPath);
if (logError) {
Log.e(TAG, "path=%s", assetPath, e);
}
......
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