diff options
author | jdduke@chromium.org <jdduke@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-26 04:24:22 +0000 |
---|---|---|
committer | jdduke@chromium.org <jdduke@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-26 04:24:22 +0000 |
commit | 5945ac2267eb79b3bfe9609d9b77f6d3c854abfe (patch) | |
tree | 0ba6c21b7f9aa1304973f05a30ca2ca8e82b6894 /ui/android | |
parent | 5a7a63a4cd6fd297e7f91c1215b8adf43fffbd3e (diff) | |
download | chromium_src-5945ac2267eb79b3bfe9609d9b77f6d3c854abfe.zip chromium_src-5945ac2267eb79b3bfe9609d9b77f6d3c854abfe.tar.gz chromium_src-5945ac2267eb79b3bfe9609d9b77f6d3c854abfe.tar.bz2 |
[Android] Harden overscroll resource allocation
It appears that overscroll scenarios are not covered by existing integration
tests on Android. Add such a test while cleaning up an overscroll-related
code path for loading potentially non-existent resources.
BUG=341406
Review URL: https://codereview.chromium.org/210623002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259474 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ui/android')
-rw-r--r-- | ui/android/java/src/org/chromium/ui/gfx/BitmapHelper.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ui/android/java/src/org/chromium/ui/gfx/BitmapHelper.java b/ui/android/java/src/org/chromium/ui/gfx/BitmapHelper.java index dd04217..5c4549c 100644 --- a/ui/android/java/src/org/chromium/ui/gfx/BitmapHelper.java +++ b/ui/android/java/src/org/chromium/ui/gfx/BitmapHelper.java @@ -39,14 +39,15 @@ public class BitmapHelper { int reqHeight) { Resources res = Resources.getSystem(); int resId = res.getIdentifier(name, null, null); + if (resId == 0) return null; final BitmapFactory.Options options = new BitmapFactory.Options(); options.inJustDecodeBounds = true; BitmapFactory.decodeResource(res, resId, options); options.inSampleSize = calculateInSampleSize(options, reqWidth, reqHeight); - options.inJustDecodeBounds = false; + options.inPreferredConfig = Bitmap.Config.ARGB_8888; return BitmapFactory.decodeResource(res, resId, options); } |