Fix crash in legacy notification construction path.

dsandler · dsandler · commit cde8aae955f2 · 2012-04-04T23:46:19.000-04:00
Bug: 6293584
Change-Id: I8c1c5162a2e9e3ebc5a6d3fed045c6c6a64b9bfd
diff --git a/core/java/android/app/Notification.java b/core/java/android/app/Notification.java
@@ -809,6 +809,7 @@ public Notification[] newArray(int size)
     @Deprecated
     public void setLatestEventInfo(Context context,
             CharSequence contentTitle, CharSequence contentText, PendingIntent contentIntent) {
+        // TODO: rewrite this to use Builder
         RemoteViews contentView = new RemoteViews(context.getPackageName(),
                 R.layout.notification_template_base);
         if (this.icon != 0) {
@@ -821,6 +822,7 @@ public void setLatestEventInfo(Context context,
             contentView.setTextViewText(R.id.text, contentText);
         }
         if (this.when != 0) {
+            contentView.setViewVisibility(R.id.time, View.VISIBLE);
             contentView.setLong(R.id.time, "setTime", when);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -809,6 +809,7 @@ public Notification[] newArray(int size)`
`809`	`809`	`@Deprecated`
`810`	`810`	`public void setLatestEventInfo(Context context,`
`811`	`811`	`CharSequence contentTitle, CharSequence contentText, PendingIntent contentIntent) {`
	`812`	`+ // TODO: rewrite this to use Builder`
`812`	`813`	`RemoteViews contentView = new RemoteViews(context.getPackageName(),`
`813`	`814`	`R.layout.notification_template_base);`
`814`	`815`	`if (this.icon != 0) {`
`@@ -821,6 +822,7 @@ public void setLatestEventInfo(Context context,`
`821`	`822`	`contentView.setTextViewText(R.id.text, contentText);`
`822`	`823`	`}`
`823`	`824`	`if (this.when != 0) {`
	`825`	`+ contentView.setViewVisibility(R.id.time, View.VISIBLE);`
`824`	`826`	`contentView.setLong(R.id.time, "setTime", when);`
`825`	`827`	`}`
`826`	`828`