in layout/reftests/canvas/size-change-1.html, intermittent 1 or 2 ASSERTION: CanvasLayer: Updated rect bigger than bounds!: 'mUpdatedRect.IsEmpty() || mBounds.Contains(mUpdatedRect)'

RESOLVED FIXED in mozilla2.0b7

Status

()

defect
RESOLVED FIXED
9 years ago
7 years ago

People

(Reporter: khuey, Assigned: philor)

Tracking

({intermittent-failure})

unspecified
mozilla2.0b7
x86
macOS
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Highly suspect this is the same bug though the error is slightly different

assertion count 1 is more than expected 0 assertions
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1279930106.1279931560.27871.gz
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Summary: REFTEST TEST-UNEXPECTED-FAIL | file:///Users/cltbld/talos-slave/mozilla-central-leopard-debug-u-reftest/build/reftest/tests/layout/reftests/canvas/size-change-1.html | assertion count 2 is more than expected 0 assertions → REFTEST TEST-UNEXPECTED-FAIL | file:///Users/cltbld/talos-slave/mozilla-central-leopard-debug-u-reftest/build/reftest/tests/layout/reftests/canvas/size-change-1.html | assertion count [1|2] is more than expected 0 assertions
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
For reference, the extra single assertion is:
###!!! 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
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
(Assignee)

Updated

9 years ago
Depends on: 591859
(Assignee)

Comment 55

9 years ago
Posted patch Fix v.1 (obsolete) — Splinter Review
Not sure whether I'm not finding an example to copy other than the 10.5-only one because we don't have one, or because I don't know what the easy way to get "any Mac" is.
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Attachment #470376 - Flags: review?(dbaron)
Comment hidden (Legacy TBPL/Treeherder Robot)
Summary: REFTEST TEST-UNEXPECTED-FAIL | file:///Users/cltbld/talos-slave/mozilla-central-leopard-debug-u-reftest/build/reftest/tests/layout/reftests/canvas/size-change-1.html | assertion count [1|2] is more than expected 0 assertions → in layout/reftests/canvas/size-change-1.html, intermittent 1 or 2 ASSERTION: CanvasLayer: Updated rect bigger than bounds!: 'mUpdatedRect.IsEmpty() || mBounds.Contains(mUpdatedRect)'
Comment on attachment 470376 [details] [diff] [review]
Fix v.1

We normally write mac-specific failures as "cocoaWidget".  You're also missing a close paren.  So review- on this, but r=dbaron on "asserts-if(cocoaWidget,0-2)"
Attachment #470376 - Flags: review?(dbaron) → review-
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Vlad, can we bump the assertion count here?  It doesn't seem like we're getting any useful information here.
Well, it would be interesting to know -why- that's happening.  Not sure why we'd hit that, unless we didn't create a new canvas layer when we should have (because the size changed).
(Assignee)

Comment 82

9 years ago
I know it's a bit difficult to see with all the starring, but I do already have r+ to adjust the count, I just need tree-room to land, and I did already file bug 591859 for your quieter place to figure out what happened on July 23rd to start it happening.
(Assignee)

Comment 83

9 years ago
Posted patch For checkinSplinter Review
I'd love to claim that I botched the parens because I was concentrating so hard on getting my trychooser syntax right, but since I botched that twice too, I probably can't. This, however, works.
Attachment #470376 - Attachment is obsolete: true
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
(Assignee)

Comment 89

9 years ago
http://hg.mozilla.org/mozilla-central/rev/55f5cc760da7
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b6

Updated

9 years ago
Blocks: 438871
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.