From b6094685727d183a87e52b18e83a3f02bb1d7933 Mon Sep 17 00:00:00 2001 From: Bernardo Rufino Date: Thu, 6 Feb 2020 16:20:21 +0000 Subject: [PATCH] Mark getView() with @Nullable Since they can return null. Test: Builds Bug: 144754526 Change-Id: I46a517584d5862fc09764a9f03ac5e2603e98957 --- api/current.txt | 2 +- core/java/android/widget/Toast.java | 13 +++++++++---- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/api/current.txt b/api/current.txt index dcf23b039c075..6b8d4f4e37276 100644 --- a/api/current.txt +++ b/api/current.txt @@ -60889,7 +60889,7 @@ package android.widget { method public int getGravity(); method public float getHorizontalMargin(); method public float getVerticalMargin(); - method @Deprecated public android.view.View getView(); + method @Deprecated @Nullable public android.view.View getView(); method public int getXOffset(); method public int getYOffset(); method public static android.widget.Toast makeText(android.content.Context, CharSequence, int); diff --git a/core/java/android/widget/Toast.java b/core/java/android/widget/Toast.java index a2c70b9afbeea..8c52d1fadc523 100644 --- a/core/java/android/widget/Toast.java +++ b/core/java/android/widget/Toast.java @@ -263,9 +263,14 @@ public class Toast { /** * Return the view. * - *

Warning: Starting from Android {@link Build.VERSION_CODES#R}, for apps - * targeting API level {@link Build.VERSION_CODES#R} or higher that haven't called {@link - * #setView(View)} with a non-{@code null} view, this method will return {@code null}. + *

Toasts constructed with {@link #Toast(Context)} that haven't called {@link #setView(View)} + * with a non-{@code null} view will return {@code null} here. + * + *

Starting from Android {@link Build.VERSION_CODES#R}, in apps targeting API level {@link + * Build.VERSION_CODES#R} or higher, toasts constructed with {@link #makeText(Context, + * CharSequence, int)} or its variants will also return {@code null} here unless they had called + * {@link #setView(View)} with a non-{@code null} view. If you want to be notified when the + * toast is shown or hidden, use {@link #addCallback(Callback)}. * * @see #setView * @deprecated Custom toast views are deprecated. Apps can create a standard text toast with the @@ -276,7 +281,7 @@ public class Toast { * will not have custom toast views displayed. */ @Deprecated - public View getView() { + @Nullable public View getView() { return mNextView; }