review

Author: leotianlizhan

firebase-perf/CHANGELOG.md

@@ -22,7 +22,7 @@ Refer [GMaven](https://maven.google.com/web/index.html?q=firebase-perf#com.googl
 ## Unreleased
 
 *   {{fixed}} Fixed a bug where screen traces were not capturing frame metrics for multi-activity apps.
-*   {{feature}} Add support for out-of-the-box Fragment screen performance measurement.
+*   {{feature}} Added support for measuring screen performance metrics for "Activity Fragments" out-of-the-box.
 
 ## Released
 

firebase-perf/dev-app/src/main/AndroidManifest.xml

@@ -35,6 +35,9 @@
         <meta-data
             android:name="sessions_sampling_percentage"
             android:value="100.0" />
+        <meta-data
+            android:name="fragment_sampling_percentage"
+            android:value="100.0" />
 
         <receiver
             android:name=".FirebasePerfTestReceiver"

firebase-perf/e2e-app/src/main/AndroidManifest.xml

@@ -33,6 +33,9 @@
         <meta-data
             android:name="sessions_sampling_percentage"
             android:value="100.0" />
+        <meta-data
+            android:name="fragment_sampling_percentage"
+            android:value="100.0" />
     </application>
 
 </manifest>

firebase-perf/src/main/java/com/google/firebase/perf/config/ConfigurationConstants.java

@@ -640,7 +640,7 @@ protected static synchronized FragmentSamplingRate getInstance() {
 
     @Override
     protected Float getDefault() {
-      // Sampling rate range is [0.00f, 1.00f]. By default, sampling rate is 1.00f, which is 100%.
+      // Sampling rate range is [0.00f, 1.00f]. By default, sampling rate is 0.00f, which is 0%.
       // 0.00f means 0%, Fireperf will not capture any event for fragment trace from the device,
       // 1.00f means 100%, Fireperf will capture all events for fragment trace from the device.
       return 0.00f;