From 0d0c1775186ae88041ae74646119ee5ea2e9cf01 Mon Sep 17 00:00:00 2001
From: Griff Hazen <griff@google.com>
Date: Tue, 17 Jun 2014 00:38:38 -0700
Subject: [PATCH] [Doc fix] Document best practices for display intent
 activities.

Bug: 15412926

Change-Id: I1c3082dfde85cd03ffc97be6383a0608f925edcf
(cherry picked from commit 831ca9d3bc73e37f555f6c07babde270f83044fe)
---
 core/java/android/app/Notification.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/core/java/android/app/Notification.java b/core/java/android/app/Notification.java
index 276f936605a61..55abdb6181233 100644
--- a/core/java/android/app/Notification.java
+++ b/core/java/android/app/Notification.java
@@ -3619,14 +3619,16 @@ public class Notification implements Parcelable
          *         .build();</pre>
          *
          * <p>The activity to launch needs to allow embedding, must be exported, and
-         * should have an empty task affinity.
+         * should have an empty task affinity. It is also recommended to use the device
+         * default light theme.
          *
          * <p>Example AndroidManifest.xml entry:
          * <pre class="prettyprint">
          * &lt;activity android:name=&quot;com.example.MyDisplayActivity&quot;
          *     android:exported=&quot;true&quot;
          *     android:allowEmbedded=&quot;true&quot;
-         *     android:taskAffinity=&quot;&quot; /&gt;</pre>
+         *     android:taskAffinity=&quot;&quot;
+         *     android:theme=&quot;@android:style/Theme.DeviceDefault.Light&quot; /&gt;</pre>
          *
          * @param intent the {@link PendingIntent} for an activity
          * @return this object for method chaining