Attachments

Example of a situation where the proposed solution might be lacking.

no flags

Details

Timeline of the problematic situation

no flags

Details

Patch 0  Interface changes for b666446

no flags

Details
 Diff 
Splinter Review

Patch 0v2  Interface changes for b666446

no flags

Details
 Diff 
Splinter Review

Patch 0v3  Interface changes for b666446

no flags

Details
 Diff 
Splinter Review

Patch 0v4  Interface changes for b666446

dholbert:
review+

Details
 Diff 
Splinter Review

Patch 0v4 (v5)  Interface changes for b666446 [r=dholbert]

jaywir3:
review+
joe:
review+
bzbarsky:
superreview+

Details
 Diff 
Splinter Review

Patch 0 (v6)  Interface changes for b666446 [r=dholbert,JOEDREW!][sr=bz]

jaywir3:
review+
jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 1  Implementation for nsImageFrame (work in progress)

no flags

Details
 Diff 
Splinter Review

Patch 2  Implementation for RasterImage (work in progress)

no flags

Details
 Diff 
Splinter Review

Test Case 1  Animated Gifs

no flags

Details

Test Case 0  Original Test Case

no flags

Details

Patch 1 (v2)  Implementation for nsImageFrame (work in progress)

no flags

Details
 Diff 
Splinter Review

Patch 1 (v3)  Implementation for nsImageFrame (work in progress)

no flags

Details
 Diff 
Splinter Review

Patch 2 (v2)  Implementation for RasterImage (work in progress)

no flags

Details
 Diff 
Splinter Review

Patch 1 (v4)  Implementation for nsImageFrame

no flags

Details
 Diff 
Splinter Review

Patch 2 (v3)  Implementation for RasterImage

no flags

Details
 Diff 
Splinter Review

Patch 1 (v5)  Implementation for nsImageFrame

dholbert:
review+

Details
 Diff 
Splinter Review

Patch 2 (v4)  Implementation for RasterImage

dholbert:
review+

Details
 Diff 
Splinter Review

Patch 2 (v6)  Implementation for nsImageFrame

no flags

Details
 Diff 
Splinter Review

Patch 2 (v5)  Implementation for RasterImage [r=dholbert]

no flags

Details
 Diff 
Splinter Review

Patch 2 (v6)  Implementation for RasterImage [r=dholbert]

no flags

Details
 Diff 
Splinter Review

Patch 3 (v7)  Implementation for RasterImage [r=dholbert]

no flags

Details
 Diff 
Splinter Review

Patch 1  Implementation for nsRefreshDriver

no flags

Details
 Diff 
Splinter Review

Patch 2 (v7)  Implementation for nsImageFrame

no flags

Details
 Diff 
Splinter Review

Patch 4 (v8)  Implementation for RasterImage [r=dholbert]

no flags

Details
 Diff 
Splinter Review

Patch 2 (v8)  Implementation for nsImageFrame

no flags

Details
 Diff 
Splinter Review

Patch 0 (v6)  Interface changes for b666446 [r=dholbert,JOEDREW!][sr=bz]

jaywir3:
review+
jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 3  Implementation for nsImageBoxFrame

no flags

Details
 Diff 
Splinter Review

Patch 1 (v2)  Implementation for nsRefreshDriver

no flags

Details
 Diff 
Splinter Review

Patch 0 (v7)  Interface changes for b666446 [r=dholbert,JOEDREW!][sr=bz]

jaywir3:
review+
jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 1 (v3)  Implementation for nsRefreshDriver

no flags

Details
 Diff 
Splinter Review

Patch 2 (v8)  Implementation for nsImageFrame

no flags

Details
 Diff 
Splinter Review

Patch 3 (v2)  Implementation for nsImageBoxFrame

roc:
review+

Details
 Diff 
Splinter Review

Patch 5 (v9) Implementation for RasterImage [r=dholbert]

joe:
review+

Details
 Diff 
Splinter Review

Patch 2 (v9)  Implementation for nsImageFrame

no flags

Details
 Diff 
Splinter Review

Patch 3 (v3)  Implementation for nsImageBoxFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 2 (v10)  Implementation for nsImageFrame

roc:
review+

Details
 Diff 
Splinter Review

Patch 1 (v4)  Implementation for nsRefreshDriver

no flags

Details
 Diff 
Splinter Review

Patch 1 (v5)  Implementation for nsRefreshDriver

roc:
review+

Details
 Diff 
Splinter Review

Patch 4 Implementation for nsBulletFrame

no flags

Details
 Diff 
Splinter Review

Test Case 2  Animated Bullets

no flags

Details

Patch 9 (v9)  Implementation for RasterImage [r=dholbert,JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 7  Implementation for nsImageLoader

roc:
review+

Details
 Diff 
Splinter Review

Patch 6 (v2)  Implementation for nsBulletFrame

roc:
review+

Details
 Diff 
Splinter Review

Patch 5 (v5)  Implementation for nsImageBoxFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 4 (v9)  Implementation for nsImageFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 4 (v9)  Implementation for nsImageFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 3  Implementation for nsLayoutUtils

roc:
review+

Details
 Diff 
Splinter Review

Patch 2 (v3)  Implementation for nsRefreshDriver [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 1 (v7)  Interface changes for b666446 [r=dholbert,JOEDREW!][sr=bz]

jaywir3:
review+
jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 4 (v10)  Implementation for nsImageFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 3 (v2)  Implementation for nsLayoutUtils [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 8  Implementation for nsSVGImageFrame

roc:
review+

Details
 Diff 
Splinter Review

Mochitest 1  Basic Test for Functionality of Animated GIF Images

no flags

Details
 Diff 
Splinter Review

Patch 11 (v10)  Implementation for RasterImage [r=dholbert,JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 9  Implementation for xul tree

no flags

Details
 Diff 
Splinter Review

Patch 10  Implementation of nsSVGFEImageElement

no flags

Details
 Diff 
Splinter Review

Patch 9 (v2)  Implementation for xul tree

no flags

Details
 Diff 
Splinter Review

Patch 3 (v3)  Implementation for nsLayoutUtils [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 10 (v2)  Implementation of nsSVGFEImageElement

no flags

Details
 Diff 
Splinter Review

Mochitest 2  Test for functionality of nsSVGImageFrame component

roc:
review+

Details
 Diff 
Splinter Review

Mochitest 1  Basic Test for Functionality of Animated GIF Images

no flags

Details
 Diff 
Splinter Review

Case where the SVG image filter works fine

no flags

Details

Case where the SVG image filter doesn't animate

no flags

Details

Case where the SVG image filter works fine

no flags

Details

Case where the SVG image filter doesn't animate (with 1second delay)

no flags

Details

Patch 1 (v8)  Interface changes for b666446 [r=dholbert,JOEDREW!][sr=bz]

jaywir3:
review+
jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 2 (v3)  Implementation for nsRefreshDriver [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 3 (v4)  Implementation for nsLayoutUtils [r=roc]

roc:
review+

Details
 Diff 
Splinter Review

Patch 4 (v11)  Implementation for nsImageFrame [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 5 (v6)  Implementation for nsImageBoxFrame [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 6 (v3)  Implementation for nsBulletFrame

no flags

Details
 Diff 
Splinter Review

Patch 7 (v2)  Implementation for nsImageLoader

no flags

Details
 Diff 
Splinter Review

Patch 8 (v2)  Implementation for nsSVGImageFrame

no flags

Details
 Diff 
Splinter Review

Patch 9 (v3)  Implementation for xul tree

no flags

Details
 Diff 
Splinter Review

Patch 10 (v3)  Implementation of nsSVGFEImageElement

no flags

Details
 Diff 
Splinter Review

Patch 11 (v10)  Implementation for RasterImage [r=dholbert,JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 0  Test framework for all following mochitests.

no flags

Details
 Diff 
Splinter Review

Mochitest 0  Test framework for all following mochitests.

no flags

Details
 Diff 
Splinter Review

Mochitest 1  Basic Test for Functionality of Animated GIF Images

no flags

Details
 Diff 
Splinter Review

Mochitest 2  Test for functionality of nsSVGImageFrame component

no flags

Details
 Diff 
Splinter Review

Mochitest 3  Functionality test for animated images in bullets.

no flags

Details
 Diff 
Splinter Review

Patch 1 (v8)  Interface changes for b666446 [r=dholbert,JOEDREW!][sr=bz]

jaywir3:
review+
jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 2 (v3)  Implementation for nsRefreshDriver [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 3 (v4)  Implementation for nsLayoutUtils [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 4 (v11)  Implementation for nsImageFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 5 (v6)  Implementation for nsImageBoxFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 6 (v4)  Implementation for nsBulletFrame

no flags

Details
 Diff 
Splinter Review

Patch 7 (v3)  Implementation for nsImageLoader

no flags

Details
 Diff 
Splinter Review

Patch 8 (v3)  Implementation for nsSVGImageFrame

no flags

Details
 Diff 
Splinter Review

Patch 9 (v4)  Implementation for xul tree

no flags

Details
 Diff 
Splinter Review

Patch 10 (v4)  Implementation of nsSVGFEImageElement

no flags

Details
 Diff 
Splinter Review

Patch 11 (v11)  Implementation for RasterImage [r=dholbert,JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 10 (v5)  Implementation of nsSVGFEImageElement

no flags

Details
 Diff 
Splinter Review

Mochitest 0  Test framework for all following mochitests.

no flags

Details
 Diff 
Splinter Review

Mochitest 1  Basic Test for Functionality of Animated GIF Images

no flags

Details
 Diff 
Splinter Review

Mochitest 2  Test for functionality of nsSVGImageFrame component

no flags

Details
 Diff 
Splinter Review

Mochitest 3  Functionality test for animated images in bullets.

no flags

Details
 Diff 
Splinter Review

Mochitest 4  Test for animations in background images

no flags

Details
 Diff 
Splinter Review

Mochitest 5  Test for animations in SVG filters.

no flags

Details
 Diff 
Splinter Review

Mochitest 6  Test for animations in XUL tree objects.

no flags

Details
 Diff 
Splinter Review

Mochitest 0  Test framework for all following mochitests.

no flags

Details
 Diff 
Splinter Review

Mochitest 1  Basic Test for Functionality of Animated GIF Images.

no flags

Details
 Diff 
Splinter Review

Mochitest 1  Basic Test for Functionality of Animated GIF Images.

joe:
review+

Details
 Diff 
Splinter Review

Mochitest 2  Test for functionality of nsSVGImageFrame component.

joe:
review+

Details
 Diff 
Splinter Review

Mochitest 3  Functionality test for animated images in bullets.

joe:
review+

Details
 Diff 
Splinter Review

Mochitest 4  Test for animations in background images.

joe:
review+

Details
 Diff 
Splinter Review

Mochitest 5  Test for animations in SVG filters.

joe:
review+

Details
 Diff 
Splinter Review

Mochitest 6  Test for animations in XUL tree objects.

joe:
review+

Details
 Diff 
Splinter Review

Patch 3 (v5)  Implementation for nsLayoutUtils [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 4  Implementation for nsImageLoadingContent

no flags

Details
 Diff 
Splinter Review

Patch 6 (v4)  Implementation for nsBulletFrame

no flags

Details
 Diff 
Splinter Review

Mochitest 7  Test for animations where the source reference changes midanimation.

joe:
review

Details
 Diff 
Splinter Review

Mochitest 8  Test for an animated image in an (initially) undisplayed iframe.

joe:
review+

Details
 Diff 
Splinter Review

Patch 1 (v9)  Interface changes for b666446 [r=dholbert,JOEDREW!][sr=bz]

jaywir3:
review+
jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 2 (v4)  Implementation for nsRefreshDriver [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 3 (v5)  Implementation for nsLayoutUtils [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 4 (v2)  Implementation for nsImageLoadingContent

no flags

Details
 Diff 
Splinter Review

Patch 5 (v7)  Implementation for nsImageBoxFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 6 (v5)  Implementation for nsBulletFrame

no flags

Details
 Diff 
Splinter Review

Patch 7 (v4)  Implementation for nsImageLoader

no flags

Details
 Diff 
Splinter Review

Patch 8 (v5)  Implementation for xultree

no flags

Details
 Diff 
Splinter Review

Patch 9 (v11)  Implementation for RasterImage [r=dholbert,JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 4 (v3)  Implementation for nsImageLoadingContent

no flags

Details
 Diff 
Splinter Review

Mochitest 0 (v2)  Test framework for all following mochitests.

joe:
review+

Details
 Diff 
Splinter Review

Patch 4 (v4)  Implementation for nsImageLoadingContent

roc:
review+

Details
 Diff 
Splinter Review

Patch 3 (v6)  Implementation for nsLayoutUtils [r=roc]

roc:
review+

Details
 Diff 
Splinter Review

Patch 4 (v5)  Implementation for nsImageLoadingContent [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 7 (v5)  Implementation for nsImageLoader [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 8 (v6)  Implementation for xultree

no flags

Details
 Diff 
Splinter Review

Patch 8 (v7)  Implementation for xultree

roc:
review+

Details
 Diff 
Splinter Review

Patch 6 (v6)  Implementation for nsBulletFrame

no flags

Details
 Diff 
Splinter Review

Patch 6 (v7)  Implementation for nsBulletFrame

no flags

Details
 Diff 
Splinter Review

Mochitest 0 (v2)  Test framework for all following mochitests. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 1 (v2)  Basic Test for Functionality of Animated GIF Images [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 5 (v2)  Test for animations in SVG filters. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 8 (v2)  Test for an animated image in an (initially) undisplayed iframe. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 4 (v2)  Test for animations in background images. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 5 (v3)  Test for animations in SVG filters. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 6 (v2)  Test for animations in XUL tree objects. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 7 (v6)  Implementation for nsImageLoader [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 8 (v8)  Implementation for xultree [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 6 (v8)  Implementation for nsBulletFrame

no flags

Details
 Diff 
Splinter Review

Mochitest 7 (v2)  Test for animations where the source reference changes midanimation.

joe:
review+

Details
 Diff 
Splinter Review

Mochitest 0 (v3)  Test framework for all following mochitests. [r=JOEDREW!]

joe:
review+

Details
 Diff 
Splinter Review

Patch 4 (v6)  Implementation for nsImageLoadingContent [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 5 (v7)  Implementation for nsImageBoxFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 8 (v9)  Implementation for xultree [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 0 (v4)  Test framework for all following mochitests. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 6 (v9)  Implementation for nsBulletFrame.

roc:
review+

Details
 Diff 
Splinter Review

Patch 1 (v10)  Interface changes for b666446 [r=dholbert,JOEDREW!][sr=bz]

jaywir3:
review+
jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 2 (v5)  Implementation for nsRefreshDriver [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 3 (v7)  Implementation for nsLayoutUtils [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 4 (v5)  Implementation for nsImageLoadingContent [r=roc]

jaywir3:
review+
mats:
superreview+

Details
 Diff 
Splinter Review

Patch 5 (v8)  Implementation for nsImageBoxFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 6 (v10)  Implementation for nsBulletFrame [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 7 (v7)  Implementation for nsImageLoader [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 8 (v10)  Implementation for xultree [r=roc]

jaywir3:
review+

Details
 Diff 
Splinter Review

Patch 9 (v12)  Implementation for RasterImage [r=dholbert,JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 0 (v5)  Test framework for all following mochitests. [r=JOEDREW]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 1 (v3)  Basic Test for Functionality of Animated GIF Images [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 2 (v2)  Test for functionality of nsSVGImageFrame component [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 3 (v2)  Functionality test for animated images in bullets. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 4 (v3)  Test for animations in background images. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 5 (v4)  Test for animations in SVG filters. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

Mochitest 6 (v3)  Test for animations in XUL tree objects. [r=JOEDREW!]

jaywir3:
review+

Details
 Diff 
Splinter Review

UPDATE 1  Make the nsIImageLoadingContent methods frameCreated and frameDestroyed inaccessible from script.

dholbert:
review+

Details
 Diff 
Splinter Review

Standalone Talos results

no flags

Details

Patch 10  Combined mochitests to test functionality for this bug. [r=joe]

jaywir3:
review+

Details
 Diff 
Splinter Review

Paints Plotted

no flags

Details

Patch 7 (v8)  Implementation for nsImageLoader, with fixes for performance regression

no flags

Details
 Diff 
Splinter Review

Patch 7 (v9)  Implementation for nsImageLoader, with fixes for performance regression

roc:
review+

Details
 Diff 
Splinter Review

Patch 1 (v11)  Interface changes for b666446 [r=dholbert,joe][sr=bz]

jaywir3:
review+
jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 4 (v6)  Implementation for nsImageLoadingContent. [r=roc][sr=mats]

jaywir3:
review+
jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 3 (v8)  Implementation for nsLayoutUtils.

no flags

Details
 Diff 
Splinter Review

Patch 1 (v12)  Interface changes for b666446 [r=dholbert,joe][sr=mats]

joe:
review+
mats:
superreview+

Details
 Diff 
Splinter Review

Patch 9 (v13)  Implementation for RasterImage.

joe:
review+

Details
 Diff 
Splinter Review

Patch 3 (v9)  Implementation for nsLayoutUtils. [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 4 (v7)  Implementation for nsImageLoadingContent. [r=roc][sr=mats]

jaywir3:
superreview+

Details
 Diff 
Splinter Review

Patch 5 (v9)  Implementation for nsImageBoxFrame. [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 6 (v11)  Implementation for nsBulletFrame. [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 7 (v10)  Implementation for nsImageLoader. [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 8 (v11)  Implementation for xultree. [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 3 (v10)  Implementation for nsLayoutUtils. [r=roc]

no flags

Details
 Diff 
Splinter Review

Patch 3 (v11)  Implementation for nsLayoutUtils. [r=roc]

roc:
review+

Details
 Diff 
Splinter Review

Patch 4 (v8)  Implementation for nsImageLoadingContent. [r=roc][sr=mats]

no flags

Details
 Diff 
Splinter Review

Patch 4 (v9)  Implementation for nsImageLoadingContent. [r=roc][sr=mats]

roc:
review+

Details
 Diff 
Splinter Review

Patch 5 (v10)  Implementation for nsImageBoxFrame. [r=roc]

roc:
review+

Details
 Diff 
Splinter Review

Patch 6 (v12)  Implementation for nsBulletFrame. [r=roc]

roc:
review+

Details
 Diff 
Splinter Review

Patch 7 (v11)  Implementation for nsImageLoader. [r=roc]

roc:
review+

Details
 Diff 
Splinter Review

Patch 8 (v12)  Implementation for xultree. [r=roc]

roc:
review+

Details
 Diff 
Splinter Review

Talos Runs

no flags

Details
