591859 - ###!!! ASSERTION: CanvasLayer: Updated rect bigger than bounds!: 'mUpdatedRect.IsEmpty() || mBounds.Contains(mUpdatedRect)' in reftest/tests/layout/reftests/canvas/size-change-1.html

Status: RESOLVED WORKSFORME

(Core :: Graphics: Canvas2D, defect)

Description

[Phil Ringnalda (:philor)]

Starting with http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1279923646.1279925056.31725.gz on 2010/07/23 15:20:46, Mac reftests on 10.5 and 10.6 have been intermittently hitting

REFTEST TEST-START | file:///Users/cltbld/talos-slave/mozilla-central_leopard-debug_test-reftest/build/reftest/tests/layout/reftests/canvas/size-change-1.html
WARNING: Application is active but there is no active document: file /builds/slave/mozilla-central-macosx-debug/build/widget/src/cocoa/nsCocoaTextInputHandler.mm, line 737
++DOMWINDOW == 51 (0xad87328) [serial = 5446] [outer = 0xdc870e0]
WARNING: Application is active but there is no active document: file /builds/slave/mozilla-central-macosx-debug/build/widget/src/cocoa/nsCocoaTextInputHandler.mm, line 737
WARNING: Application is active but there is no active document: file /builds/slave/mozilla-central-macosx-debug/build/widget/src/cocoa/nsCocoaTextInputHandler.mm, line 737
WARNING: Application is active but there is no active document: file /builds/slave/mozilla-central-macosx-debug/build/widget/src/cocoa/nsCocoaTextInputHandler.mm, line 737
WARNING: Application is active but there is no active document: file /builds/slave/mozilla-central-macosx-debug/build/widget/src/cocoa/nsCocoaTextInputHandler.mm, line 737
###!!! ASSERTION: CanvasLayer: Updated rect bigger than bounds!: 'mUpdatedRect.IsEmpty() || mBounds.Contains(mUpdatedRect)', file /builds/slave/mozilla-central-macosx-debug/build/gfx/layers/basic/BasicLayers.cpp, line 734
mozilla::layers::BasicCanvasLayer::Updated [gfx/layers/basic/BasicLayers.cpp:735]
nsCanvasRenderingContext2D::GetCanvasLayer [content/canvas/src/nsCanvasRenderingContext2D.cpp:4161]
nsHTMLCanvasElement::GetCanvasLayer [content/html/content/src/nsHTMLCanvasElement.cpp:545]
nsHTMLCanvasFrame::BuildLayer [layout/generic/nsHTMLCanvasFrame.cpp:253]
nsDisplayCanvas::BuildLayer [layout/generic/nsHTMLCanvasFrame.cpp:96]
mozilla::::ContainerState::ProcessDisplayItems [layout/base/FrameLayerBuilder.cpp:948]
mozilla::::ContainerState::ProcessDisplayItems [layout/base/FrameLayerBuilder.cpp:920]
mozilla::FrameLayerBuilder::BuildContainerLayerFor [layout/base/FrameLayerBuilder.cpp:1232]
nsDisplayOwnLayer::BuildLayer [layout/base/nsDisplayList.cpp:1280]
mozilla::::ContainerState::ProcessDisplayItems [layout/base/FrameLayerBuilder.cpp:948]
mozilla::::ContainerState::ProcessDisplayItems [layout/base/FrameLayerBuilder.cpp:920]
mozilla::FrameLayerBuilder::BuildContainerLayerFor [layout/base/FrameLayerBuilder.cpp:1232]
nsDisplayList::PaintForFrame [layout/base/nsDisplayList.cpp:399]
nsDisplayList::PaintRoot [layout/base/nsDisplayList.cpp:351]
nsLayoutUtils::PaintFrame [layout/base/nsLayoutUtils.cpp:1412]
PresShell::Paint [layout/base/nsPresShell.cpp:5936]
nsViewManager::RenderViews [view/src/nsViewManager.cpp:459]
nsViewManager::Refresh [view/src/nsViewManager.cpp:427]
nsViewManager::DispatchEvent [view/src/nsViewManager.cpp:941]
HandleEvent [view/src/nsView.cpp:161]
nsChildView::DispatchEvent [widget/src/cocoa/nsChildView.mm:1715]
nsChildView::DispatchWindowEvent [widget/src/cocoa/nsChildView.mm:1726]
-[ChildView drawRect:inContext:] [widget/src/cocoa/nsChildView.mm:2617]
-[ChildView drawRect:] [widget/src/cocoa/nsChildView.mm:2526]
AppKit + 0x108bf8
AppKit + 0x106469
AppKit + 0x106e9f
AppKit + 0x106e9f
AppKit + 0x105987
AppKit + 0x1024ab
AppKit + 0x42e7b
AppKit + 0x42a29
AppKit + 0x4284c
CoreFoundation + 0x71772
CoreFoundation + 0x72acc
CoreFoundation + 0x73aa8
HIToolbox + 0x302ac
HIToolbox + 0x2fffe
HIToolbox + 0x2ff39
AppKit + 0x406d5
AppKit + 0x3ff88
AppKit + 0x38f9f
nsAppShell::Run [widget/src/cocoa/nsAppShell.mm:747]
nsAppStartup::Run [toolkit/components/startup/src/nsAppStartup.cpp:191]
XRE_main [toolkit/xre/nsAppRunner.cpp:3662]
main [browser/app/nsBrowserApp.cpp:158]
firefox-bin + 0x148e

(and then bitching about the expected assertion count, to be fixed in bug 581592)

Comment 1

[Phil Ringnalda (:philor)]

Oh, and http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1282765981.1282767377.25680.gz&fulltext=1#err0 for the rarer case where both the test and the reference fail the assertion.

Comment 2

[Phil Ringnalda (:philor)]

And then somewhere along the line, someone decided to remove that assertion.