<div dir="ltr"><div>Great idea, I'll add that to the list! Times actually are on that page too, I just didn't know about performance.now() as an alternative.<br></div><div><br></div><div>Also, it was pointed out to me that in order to answer question 1) it would help to know how to disable these rules. Eslint rules can be disabled at the line or file level via comments. They can also be disabled at the directory level in .eslintrc.js. So anyone wanting to legitimately use these patterns would need to figure out the most appropriate way to disable the corresponding rule.<br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 6, 2017 at 3:45 PM, Chris Peterson <span dir="ltr"><<a href="mailto:cpeterson@mozilla.com" target="_blank">cpeterson@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF"><span class="">
    <br>
    <br>
    <div class="m_-8582273285194302015moz-cite-prefix">On 7/6/17 11:47 AM, Andrew Halberstadt
      wrote:<br>
    </div>
    <blockquote type="cite">
      <li>Are there additional things not listed on that page that we
        could lint for?</li>
    </blockquote>
    <br></span>
    Do we want to discourage tests from using Date (`new Date` or
    `Date.now()`) for measuring time? Dates are affected by time zones,
    DST, and clock skew issues jumping forward or backward. The
    performance.now() API doesn't have any of those problems, plus it
    uses high-resolution timestamps with 5 microsecond resolution
    instead of 1 millisecond. In bug 1372261, mconley is changing the
    tps Talos test to use performance.timing.<wbr>navigationStart +
    performance.now().<br>
  </div>

</blockquote></div><br></div>