Bug 100282

Summary: Fix ASSERT(!currBox->needsLayout()) triggering on gmail
Product: WebKit Reporter: Ami Fischman <fischman>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: jchaffraix, leviw, mad, mihnea, simon.fraser, webkit
Priority: P2 Keywords: GoogleBug, NeedsReduction
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   

Description Ami Fischman 2012-10-24 13:00:12 PDT
Originally reported in bug 89733, comment 10:

ASSERTION FAILED: !currBox->needsLayout()             
../../third_party/WebKit/Source/WebCore/rendering/RenderBlock.cpp(7671) : void WebCore::RenderBlock::checkPositionedObjectsNeedLayout()
1   0x7f1e524abb40                                    
2   0x7f1e52601519                                    
3   0x7f1e5245152b
4   0x7f1e5248933a
5   0x7f1e52488293
6   0x7f1e524877ae
7   0x7f1e52687bb9
8   0x7f1e526882cc
9   0x7f1e5348f60e
10  0x7f1e53498645
11  0x7f1e520236fe
12  0x7f1e520e8014
13  0x7f1e4fb7cf75 content::RenderWidget::DoDeferredUpdate()
14  0x7f1e4fb82319 content::RenderWidget::DoDeferredUpdateAndSendInputAck()
15  0x7f1e4fb83a7a content::RenderWidget::InvalidationCallback()
16  0x7f1e4fb8a6d2
17  0x7f1e4fb8b5a9
18  0x7f1e4fb8b565
19  0x7f1e5602ccfe
20  0x7f1e560770ec MessageLoop::RunTask(base::PendingTask const&)
21  0x7f1e560774db MessageLoop::DeferOrRunPendingTask(base::PendingTask const&)
22  0x7f1e56077685 MessageLoop::DoWork()
23  0x7f1e56081408
24  0x7f1e56076b16 MessageLoop::RunInternal()
25  0x7f1e560769c5 MessageLoop::RunHandler()
26  0x7f1e560b3552 base::RunLoop::Run()
27  0x7f1e56076261 MessageLoop::Run()
28  0x7f1e4fb9c390
29  0x7f1e4f405987
30  0x7f1e4f405c84
31  0x7f1e4f406b0d

Using gmail in a Debug chrome for a few minutes triggers this tab crash.
Probably needs a fix similar to bug 100265.
Comment 1 Julien Chaffraix 2012-10-25 04:37:53 PDT
The ASSERT was added in bug 88133 but it probably just uncovered an existing issue.
Comment 2 Levi Weintraub 2012-10-25 04:48:34 PDT
(In reply to comment #1)
> The ASSERT was added in bug 88133 but it probably just uncovered an existing issue.

Can you CC me on that bug?