commit 831ca9d3bc73e37f555f6c07babde270f83044fe
author Griff Hazen <griff@google.com> Tue Jun 17 00:38:38 2014 -0700
committer Griff Hazen <griff@google.com> Tue Jun 17 00:38:38 2014 -0700
tree 015010eed09f3a7d46c9d8c4bd5bd476958304b6
parent b17420b81ebe7e1235a45ad3e7ce8c1850c129f2

Document best practices for display intent activities.

Bug: 15412926

Change-Id: I1c3082dfde85cd03ffc97be6383a0608f925edcf

core/java/android/app/Notification.java

diff --git a/core/java/android/app/Notification.java b/core/java/android/app/Notification.java
index 32284e8..2ea679d 100644
--- a/core/java/android/app/Notification.java
+++ b/core/java/android/app/Notification.java
@@ -3102,14 +3102,16 @@
          *         .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