diff options
author | Michael Jurka <mikejurka@google.com> | 2012-09-04 03:49:04 -0700 |
---|---|---|
committer | Michael Jurka <mikejurka@google.com> | 2012-09-04 03:52:16 -0700 |
commit | a4dd30174d8f0b385cfb37258c583afcb725794c (patch) | |
tree | 16a24e6be4f7b1dc44dc61b0f62676e891924dbd | |
parent | e94831e5ed4aa3b1ebc5444cc1b16cdab8cb92f8 (diff) | |
download | frameworks_base-a4dd30174d8f0b385cfb37258c583afcb725794c.zip frameworks_base-a4dd30174d8f0b385cfb37258c583afcb725794c.tar.gz frameworks_base-a4dd30174d8f0b385cfb37258c583afcb725794c.tar.bz2 |
Debug code for window transition crash
https://b.corp.google.com/7060284
- Throw exception before we try to start window
transition with null thumb
- also remove some old unneeded debug code
Change-Id: Ic9693372724cbfaa4c4cb2594128ca10677d9d8f
-rw-r--r-- | packages/SystemUI/src/com/android/systemui/recent/RecentsPanelView.java | 10 | ||||
-rw-r--r-- | packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java | 10 |
2 files changed, 11 insertions, 9 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/recent/RecentsPanelView.java b/packages/SystemUI/src/com/android/systemui/recent/RecentsPanelView.java index c3ecdb5..b133ff9 100644 --- a/packages/SystemUI/src/com/android/systemui/recent/RecentsPanelView.java +++ b/packages/SystemUI/src/com/android/systemui/recent/RecentsPanelView.java @@ -164,13 +164,6 @@ public class RecentsPanelView extends FrameLayout implements OnItemClickListener public View getView(int position, View convertView, ViewGroup parent) { if (convertView == null) { convertView = createView(parent); - if (convertView.getParent() != null) { - throw new RuntimeException("Recycled child has parent"); - } - } else { - if (convertView.getParent() != null) { - throw new RuntimeException("Recycled child has parent"); - } } ViewHolder holder = (ViewHolder) convertView.getTag(); @@ -600,6 +593,9 @@ public class RecentsPanelView extends FrameLayout implements OnItemClickListener usingDrawingCache = true; } + if (bm == null) { + throw new RuntimeException("Recents thumbnail is null"); + } ActivityOptions opts = ActivityOptions.makeThumbnailScaleUpAnimation( holder.thumbnailViewImage, bm, 0, 0, null); diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java b/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java index 106ce7e..832cf88 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java @@ -488,11 +488,18 @@ public abstract class BaseStatusBar extends SystemUI implements .getDimensionPixelSize(R.dimen.status_bar_recents_thumbnail_width); float thumbHeight = res .getDimensionPixelSize(R.dimen.status_bar_recents_thumbnail_height); + if (first == null) { + throw new RuntimeException("Recents thumbnail is null"); + } if (first.getWidth() != thumbWidth || first.getHeight() != thumbHeight) { first = Bitmap.createScaledBitmap(first, (int) thumbWidth, (int) thumbHeight, true); + if (first == null) { + throw new RuntimeException("Recents thumbnail is null"); + } } + DisplayMetrics dm = new DisplayMetrics(); mDisplay.getMetrics(dm); // calculate it here, but consider moving it elsewhere @@ -521,8 +528,7 @@ public abstract class BaseStatusBar extends SystemUI implements + thumbBgPadding + thumbLeftMargin); y = (int) (dm.heightPixels - res.getDimensionPixelSize(R.dimen.status_bar_recents_thumbnail_height) - thumbBgPadding); - } else { // if (config.orientation == - // Configuration.ORIENTATION_LANDSCAPE) { + } else { // if (config.orientation == Configuration.ORIENTATION_LANDSCAPE) { float thumbTopMargin = res .getDimensionPixelSize(R.dimen.status_bar_recents_thumbnail_top_margin); float thumbBgPadding = res |