Merge "Add null check for quick share intent" into tm-qpr-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
81a1b8bb85
@@ -825,12 +825,23 @@ public class ScreenshotView extends FrameLayout implements
|
|||||||
}
|
}
|
||||||
});
|
});
|
||||||
if (mQuickShareChip != null) {
|
if (mQuickShareChip != null) {
|
||||||
|
if (imageData.quickShareAction != null) {
|
||||||
mQuickShareChip.setPendingIntent(imageData.quickShareAction.actionIntent,
|
mQuickShareChip.setPendingIntent(imageData.quickShareAction.actionIntent,
|
||||||
() -> {
|
() -> {
|
||||||
mUiEventLogger.log(
|
mUiEventLogger.log(
|
||||||
ScreenshotEvent.SCREENSHOT_SMART_ACTION_TAPPED, 0, mPackageName);
|
ScreenshotEvent.SCREENSHOT_SMART_ACTION_TAPPED, 0,
|
||||||
|
mPackageName);
|
||||||
animateDismissal();
|
animateDismissal();
|
||||||
});
|
});
|
||||||
|
} else {
|
||||||
|
// hide chip and unset pending interaction if necessary, since we don't actually
|
||||||
|
// have a useable quick share intent
|
||||||
|
Log.wtf(TAG, "Showed quick share chip, but quick share intent was null");
|
||||||
|
if (mPendingInteraction == PendingInteraction.QUICK_SHARE) {
|
||||||
|
mPendingInteraction = null;
|
||||||
|
}
|
||||||
|
mQuickShareChip.setVisibility(GONE);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (mPendingInteraction != null) {
|
if (mPendingInteraction != null) {
|
||||||
|
|||||||
Reference in New Issue
Block a user