Merge pull request #14467 from hfiguiere/bug819080-perf

Bug 819080 - Add performance testing for the fmradio HAL r=julienw

apps/fm/index.html

@@ -8,6 +8,10 @@
     <link rel="resource" type="application/l10n" href="locales/locales.ini" />
     <script defer src="shared/js/l10n.js"></script>
     <script defer src="shared/js/async_storage.js"></script>
+
+    <!-- For perf-measurement related utilities -->
+    <script defer src="shared/js/performance_testing_helper.js"></script>
+
     <script defer src="js/fm.js"></script>
 
   </head>

apps/fm/js/fm.js

@@ -187,9 +187,14 @@ function updateFreqUI() {
 }
 
 function updatePowerUI() {
-  console.log('Power status: ' + (mozFMRadio.enabled ? 'on' : 'off'));
+  var enabled = mozFMRadio.enabled;
+  if (enabled) {
+    PerformanceTestingHelper.dispatch('fm-radio-enabled');
+    PerformanceTestingHelper.dispatch('startup-path-done');
+  }
+  console.log('Power status: ' + (enabled ? 'on' : 'off'));
   var powerSwitch = $('power-switch');
-  powerSwitch.dataset.enabled = mozFMRadio.enabled;
+  powerSwitch.dataset.enabled = enabled;
   powerSwitch.dataset.enabling = enabling;
 }
 
@@ -724,6 +729,8 @@ var favoritesList = {
 };
 
 function init() {
+  PerformanceTestingHelper.dispatch('start');
+
   frequencyDialer.init();
 
   var seeking = false;

apps/fm/test/unit/fm_test.js

@@ -1,5 +1,10 @@
 requireApp('fm/js/fm.js');
 
+var PerformanceTestingHelper = {
+  dispatch: function() { }
+};
+
+
 suite('FM', function() {
   var tempNode;
 

tests/performance/startup_events_test.js

@@ -6,12 +6,12 @@ var App = require('./app');
 var PerformanceHelper = requireGaia('/tests/performance/performance_helper.js');
 var MarionetteHelper = requireGaia('/tests/js-marionette/helper.js');
 
-// This test is only for communications/contacts for now.
-// XXX extend to more apps.
 var whitelistedApps = [
   'communications/contacts',
-  'clock'
+  'clock',
+  'fm'
 ];
+
 if (whitelistedApps.indexOf(mozTestInfo.appPath) === -1) {
   return;
 }