From 9e3375fb45f5f852f19f66d1a8d5e25e74dd3d51 Mon Sep 17 00:00:00 2001 From: Jonathan Watt Date: Thu, 16 May 2013 01:53:03 +0100 Subject: [PATCH] Bug 872619 - Stop using 'load' in many SVG reftests, and use MozReftestInvalidate instead, and flag the fallback setTimeouts. r=dholbert --HG-- extra : rebase_source : c41d8fb0048e6c6eea70db12d5b6e976fc442b69 --- layout/reftests/svg/as-image/svg-image-script-2.svg | 3 +-- .../reftests/svg/as-image/svg-image-visited-1.html | 12 +++++++----- .../reftests/svg/as-image/svg-image-visited-2.html | 12 +++++++----- layout/reftests/svg/dynamic-clipPath-01.svg | 3 +-- layout/reftests/svg/dynamic-clipPath-02.svg | 3 +-- layout/reftests/svg/dynamic-filter-contents-01a.svg | 3 +-- layout/reftests/svg/dynamic-filter-contents-01b.svg | 3 +-- layout/reftests/svg/dynamic-gradient-contents-01.svg | 3 +-- layout/reftests/svg/dynamic-gradient-contents-02.svg | 3 +-- layout/reftests/svg/dynamic-inner-svg-01.svg | 3 +-- layout/reftests/svg/dynamic-marker-01.svg | 3 +-- layout/reftests/svg/dynamic-marker-02.svg | 3 +-- layout/reftests/svg/dynamic-marker-03.svg | 3 +-- layout/reftests/svg/dynamic-mask-01.svg | 3 +-- layout/reftests/svg/dynamic-mask-contents-01.svg | 3 +-- layout/reftests/svg/dynamic-pattern-01.svg | 3 +-- layout/reftests/svg/dynamic-pattern-02.svg | 9 +++------ layout/reftests/svg/dynamic-pattern-contents-01.svg | 9 +++------ layout/reftests/svg/dynamic-pattern-contents-02.svg | 9 +++------ .../svg/dynamic-small-object-scaled-up-01.svg | 4 +--- .../svg/dynamic-small-object-scaled-up-02.svg | 4 +--- layout/reftests/svg/dynamic-text-04.svg | 3 +-- layout/reftests/svg/dynamic-text-05.svg | 9 ++++----- layout/reftests/svg/dynamic-text-06.svg | 4 +++- layout/reftests/svg/dynamic-textPath-01.svg | 9 +++------ layout/reftests/svg/dynamic-textPath-02.svg | 9 +++------ layout/reftests/svg/dynamic-textPath-03.svg | 9 +++------ layout/reftests/svg/dynamic-use-03.svg | 3 +-- layout/reftests/svg/dynamic-use-04.svg | 3 +-- layout/reftests/svg/dynamic-use-05.svg | 3 +-- layout/reftests/svg/dynamic-use-06.svg | 3 +-- layout/reftests/svg/filter-invalidation-01.svg | 2 +- .../filters/dynamic-filtered-foreignObject-01.svg | 2 +- .../svg/foreignObject-ancestor-style-change-01.svg | 10 +++------- .../svg/foreignObject-change-transform-01.svg | 10 +++------- layout/reftests/svg/foreignObject-display-01.svg | 11 ++++++----- .../reftests/svg/foreignObject-move-repaint-01.svg | 10 +++------- .../reftests/svg/foreignObject-start-hidden-01.svg | 10 ++++------ .../reftests/svg/foreignObject-start-hidden-02.svg | 10 ++++------ .../reftests/svg/foreignObject-style-change-01.svg | 9 +++------ .../reftests/svg/outer-svg-border-and-padding-01.svg | 2 +- .../svg/smil/anim-filter-primitive-size-01.svg | 3 +-- layout/reftests/svg/smil/anim-filter-size-01.svg | 3 +-- layout/reftests/svg/smil/anim-view-01.svg | 3 +-- layout/reftests/svg/smil/event/event-seek-1.svg | 5 +++-- .../svg/smil/event/event-target-non-svg-1.xhtml | 6 +++++- layout/reftests/svg/smil/sort/sort-startSame-1b.svg | 5 +++-- .../svg/smil/syncbase/cross-container-3.xhtml | 6 +++++- layout/reftests/svg/smil/timed/appendChild-1a.svg | 9 ++++++--- layout/reftests/svg/smil/timed/appendChild-1b.svg | 9 ++++++--- layout/reftests/svg/smil/timed/appendChild-2a.svg | 9 ++++++--- layout/reftests/svg/smil/timed/appendChild-2b.svg | 9 ++++++--- layout/reftests/svg/smil/timed/paced-1.svg | 9 ++++++--- layout/reftests/svg/smil/timed/pause-1.svg | 9 ++++++--- layout/reftests/svg/smil/timed/removeChild-1.svg | 9 ++++++--- layout/reftests/svg/smil/timed/removeChild-2.svg | 9 ++++++--- layout/reftests/svg/suspend-02.svg | 3 +-- layout/reftests/svg/suspend-04.svg | 3 +-- 58 files changed, 158 insertions(+), 183 deletions(-) diff --git a/layout/reftests/svg/as-image/svg-image-script-2.svg b/layout/reftests/svg/as-image/svg-image-script-2.svg index 4059d133278a..84aeaf1342a8 100644 --- a/layout/reftests/svg/as-image/svg-image-script-2.svg +++ b/layout/reftests/svg/as-image/svg-image-script-2.svg @@ -10,8 +10,7 @@ - - - + diff --git a/layout/reftests/svg/as-image/svg-image-visited-2.html b/layout/reftests/svg/as-image/svg-image-visited-2.html index 48110a6a578e..e1c66698c47a 100644 --- a/layout/reftests/svg/as-image/svg-image-visited-2.html +++ b/layout/reftests/svg/as-image/svg-image-visited-2.html @@ -1,6 +1,5 @@ @@ -10,12 +9,15 @@ function snapshot() { document.documentElement.removeAttribute("class"); } + function delay_snapshot() { + setTimeout(snapshot, 100); // delay snapshot to allow time for + // :visited style inside the image to + // resolve (which it hopefully won't) + } + document.addEventListener("MozReftestInvalidate", delay_snapshot, false); - - - + diff --git a/layout/reftests/svg/dynamic-clipPath-01.svg b/layout/reftests/svg/dynamic-clipPath-01.svg index 6c31a7cda5de..1d1e86ebb861 100644 --- a/layout/reftests/svg/dynamic-clipPath-01.svg +++ b/layout/reftests/svg/dynamic-clipPath-01.svg @@ -82,8 +82,7 @@ diff --git a/layout/reftests/svg/dynamic-mask-01.svg b/layout/reftests/svg/dynamic-mask-01.svg index 59cb3f050dae..7a0e428ee78a 100644 --- a/layout/reftests/svg/dynamic-mask-01.svg +++ b/layout/reftests/svg/dynamic-mask-01.svg @@ -31,8 +31,7 @@ diff --git a/layout/reftests/svg/dynamic-small-object-scaled-up-02.svg b/layout/reftests/svg/dynamic-small-object-scaled-up-02.svg index bf23045123cd..9e34ce8ab023 100644 --- a/layout/reftests/svg/dynamic-small-object-scaled-up-02.svg +++ b/layout/reftests/svg/dynamic-small-object-scaled-up-02.svg @@ -19,9 +19,7 @@ function move_small_object() } document.addEventListener("MozReftestInvalidate", move_small_object, false); - -// fallback for browsers that don't support MozReftestInvalidate -setTimeout(move_small_object, 500); +setTimeout(move_small_objects, 4000); // fallback for running outside reftest //]]> diff --git a/layout/reftests/svg/dynamic-text-04.svg b/layout/reftests/svg/dynamic-text-04.svg index 904cc513752c..b2fab69ac5cb 100644 --- a/layout/reftests/svg/dynamic-text-04.svg +++ b/layout/reftests/svg/dynamic-text-04.svg @@ -8,8 +8,7 @@ diff --git a/layout/reftests/svg/dynamic-textPath-01.svg b/layout/reftests/svg/dynamic-textPath-01.svg index 377b21214bb4..01df1b70c542 100644 --- a/layout/reftests/svg/dynamic-textPath-01.svg +++ b/layout/reftests/svg/dynamic-textPath-01.svg @@ -2,7 +2,7 @@ Any copyright is dedicated to the Public Domain. http://creativecommons.org/publicdomain/zero/1.0/ --> - + Testing that dynamic changes to the element for a given ID are reflected in textPath @@ -14,11 +14,8 @@ + Play! diff --git a/layout/reftests/svg/smil/sort/sort-startSame-1b.svg b/layout/reftests/svg/smil/sort/sort-startSame-1b.svg index e49ad6a757af..7e99e508654e 100644 --- a/layout/reftests/svg/smil/sort/sort-startSame-1b.svg +++ b/layout/reftests/svg/smil/sort/sort-startSame-1b.svg @@ -2,8 +2,7 @@ positions, the one that ends up later in the document should win. --> + class="reftest-wait"> diff --git a/layout/reftests/svg/smil/syncbase/cross-container-3.xhtml b/layout/reftests/svg/smil/syncbase/cross-container-3.xhtml index 9c41142d29a5..d5e94ac12e35 100644 --- a/layout/reftests/svg/smil/syncbase/cross-container-3.xhtml +++ b/layout/reftests/svg/smil/syncbase/cross-container-3.xhtml @@ -42,9 +42,13 @@ function snapshotB() { svga.setCurrentTime(svga.getCurrentTime()); document.documentElement.removeAttribute("class"); } + +document.addEventListener("MozReftestInvalidate", snapshot, false); +setTimeout(snapshot, 4000); // fallback for running outside reftest + - + + class="reftest-wait"> + class="reftest-wait"> + class="reftest-wait"> + class="reftest-wait"> + class="reftest-wait"> diff --git a/layout/reftests/svg/smil/timed/pause-1.svg b/layout/reftests/svg/smil/timed/pause-1.svg index f81becae640a..36101cadf042 100644 --- a/layout/reftests/svg/smil/timed/pause-1.svg +++ b/layout/reftests/svg/smil/timed/pause-1.svg @@ -3,9 +3,7 @@ a little, and make sure the animation is frozen there. --> + class="reftest-wait"> + class="reftest-wait"> + class="reftest-wait">