Bug 100282 - Fix ASSERT(!currBox->needsLayout()) triggering on gmail
Summary: Fix ASSERT(!currBox->needsLayout()) triggering on gmail
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: GoogleBug, NeedsReduction
Depends on:
Blocks:
 
Reported: 2012-10-24 13:00 PDT by Ami Fischman
Modified: 2012-11-27 10:43 PST (History)
6 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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?