author Mike Hommey <>
Fri, 02 Jun 2017 15:07:58 +0900
changeset 410640 f360dd9d411e7ba61531bed2c32a0981cc6c418a
parent 374655 7b434711753b98a87603453caaa0945791ffe12b
child 528458 ba6f655fd68963530c866d0d4a48c3db3d307777
permissions -rw-r--r--
Bug 1369622 - Restore static asserts for lack of arguments on some macros. r=froydnj Bug 1368932 removed MOZ_STATIC_ASSERT_VALID_ARG_COUNT because it didn't actually work for large numbers of arguments. But it was kind of useful for macros that expand to something broken when they are given no variadic argument at all. Now that we have a macro that doesn't require tricks to count empty arguments lists, however, we can use straightforward static_asserts instead of a generic macro, which has the side effect of providing more context in the error message.

<html xmlns="">
  <title>Tests for SMIL Reset Behavior </title>
  <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
  <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
<p id="display"></p>
<div id="content" style="display: none">
<svg id="svg" xmlns="" width="120px" height="120px"
  <circle cx="20" cy="20" r="15" fill="blue">
    <animate attributeName="cx" from="20" to="100" begin="2s" dur="4s"
             id="anim1" attributeType="XML"/>
<pre id="test">
<script class="testbody" type="text/javascript">
/** Tests for SMIL Reset Behavior  **/


function main() {
  var svg = document.getElementById("svg");
  ok(svg.animationsPaused(), "should be paused by <svg> load handler");
  is(svg.getCurrentTime(), 0, "should be paused at 0 in <svg> load handler");

  var anim = document.getElementById("anim1");
  is(anim.getStartTime(), 2, "Unexpected initial start time");


  // We now have two instance times: 2, 3

  // Restart (and reset) animation at t=1

  // Instance times should now be 1, 2 (3 should have be reset)
  is(anim.getStartTime(), 1,
    "Unexpected start time after restart. Perhaps the added instance time "
    + "was cleared");
  // Instance times will now be 2 (1 will have be reset when we restarted)
  is(anim.getStartTime(), 2, "Unexpected start time after seek");

  // Create a two new instance times at t=4, 5
  is(anim.getStartTime(), 4, "Unexpected start time after beginElement");

  // Here is a white box test to make sure we don't discard instance times
  // created by DOM calls when setting/unsetting the 'begin' spec
  is(anim.getStartTime(), 4, "Unexpected start time after clearing begin spec");
  is(anim.getStartTime(), 5,
    "Second DOM instance time cleared when begin spec was removed");

  // And likewise, when we set it again
  anim.beginElementAt(1); // Instance times now t=5s, 7s
  anim.setAttribute('begin', '1s'); // + t=1s
  is(anim.getStartTime(), 5, "Unexpected start time after setting begin spec");
  is(anim.getStartTime(), 7,
    "Second DOM instance time cleared when begin spec was added");

  // But check we do update state appropriately
  anim.setAttribute('begin', '8s');
  is(anim.getStartTime(), 8, "Interval not updated with updated begin spec");


window.addEventListener("load", main);