about:startup - hide empty message when the duration table isn't empty

toolkit/content/aboutStartup.js

@@ -78,11 +78,13 @@ var lastver, lastbuild;
 query.executeAsync({
   handleResult: function(results)
   {
+    let hasresults = false;
     for (let row = results.getNextRow(); row; row = results.getNextRow())
     {
-      var stamp = row.getResultByName("timestamp");
-      var version = row.getResultByName("appVersion");
-      var build = row.getResultByName("appBuild");
+      hasresults = true;
+      let stamp = row.getResultByName("timestamp");
+      let version = row.getResultByName("appVersion");
+      let build = row.getResultByName("appBuild");
       if (lastver != version)
       {
         options.grid.markings.push(majorMark(stamp, "Firefox "+ version +" ("+ build +")"));
@@ -93,9 +95,10 @@ query.executeAsync({
 
       lastver = version;
       lastbuild = build;
-      var l, s;
-      series[1].data.push(point(stamp, l = row.getResultByName("launch"), version, build));
-      series[0].data.push(point(stamp, l + (s = row.getResultByName("startup")), version, build));
+      let l = row.getResultByName("launch"),
+          s = row.getResultByName("startup");
+      series[1].data.push(point(stamp, l, version, build));
+      series[0].data.push(point(stamp, l + s, version, build));
       table.appendChild(tr(td(formatstamp(stamp)),
                            td(formatms(msFromµs(l))),
                            td(formatms(msFromµs(s))),
@@ -105,6 +108,8 @@ query.executeAsync({
                            td(row.getResultByName("platformVersion")),
                            td(row.getResultByName("platformBuild"))));
     }
+    if (hasresults)
+      $("#duration-table > .empty").hide();
   },
   handleError: function(error)
   {
@@ -129,16 +134,6 @@ query.executeAsync({
     $("#overview").width($("#overview").width() - offset);
     $("#overview").css("margin-left", offset);
     overview = $.plot($("#overview"), series, overviewOpts);
-
-    //var axes = graph.getAxes();
-    //overview.setSelection({ xaxis: { min: axes.xaxis.min,
-    //                                 max: axes.xaxis.max
-    //                               },
-    //                        yaxis: { min: axes.yaxis.min,
-    //                                 max: axes.yaxis.max
-    //                               }
-    //                      },
-    //                      true);
   },
 });
 

toolkit/content/aboutStartup.xhtml

@@ -33,7 +33,7 @@
     <div id="overview"/>
 
     <h1>&about.startup.table;</h1>
-    <table>
+    <table id="duration-table">
       <tr>
         <th>&about.startup.timestamp;</th>
         <th>&about.startup.duration.launch;</th>
@@ -41,11 +41,11 @@
         <th>&about.startup.duration.ready;</th>
         <th colspan="4">&about.startup.version;</th>
       </tr>
-      <tr><td colspan="8"><i>&about.startup.noevents;</i></td></tr>
+      <tr class="empty"><td colspan="8"><i>&about.startup.noevents;</i></td></tr>
     </table>
-    <table>
+    <table id="event-table">
       <tr><th>&about.startup.timestamp;</th><th>&about.startup.eventdesc;</th></tr>
-      <tr><td colspan="2"><i>&about.startup.noevents;</i></td></tr>
+      <tr class="empty"><td colspan="2"><i>&about.startup.noevents;</i></td></tr>
     </table>
     <script type="application/javascript;version=1.8" src="chrome://global/content/aboutStartup.js"/>
   </body>