Bug 94106

Summary: [Qt] Make sure that we don't prematurely force the first layout during loading.
Product: WebKit Reporter: Jocelyn Turcotte <jturcotte>
Component: New BugsAssignee: Jocelyn Turcotte <jturcotte>
Status: RESOLVED FIXED    
Severity: Normal CC: kenneth, noam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch kenneth: review+, buildbot: commit-queue-

Jocelyn Turcotte
Reported 2012-08-15 07:14:23 PDT
[Qt] Make sure that we don't prematurely force the first layout during loading.
Attachments
Patch (1.66 KB, patch)
2012-08-15 07:16 PDT, Jocelyn Turcotte 		kenneth: review+
buildbot: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jocelyn Turcotte
Comment 1 2012-08-15 07:16:35 PDT
Created attachment 158564 [details] Patch
Kenneth Rohde Christiansen
Comment 2 2012-08-15 07:17:26 PDT
Comment on attachment 158564 [details] Patch Good catch!
Jocelyn Turcotte
Comment 3 2012-08-15 07:20:52 PDT
r+ed in 51 seconds! I think it's a record :)
Kenneth Rohde Christiansen
Comment 4 2012-08-15 07:28:49 PDT
(In reply to comment #3) > r+ed in 51 seconds! > I think it's a record :) Heh! I opened the bug report before you attached the patch, so I was prepared :-)
Build Bot
Comment 5 2012-08-15 08:54:18 PDT
Comment on attachment 158564 [details] Patch Attachment 158564 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13502743
Jocelyn Turcotte
Comment 6 2012-08-16 01:20:28 PDT
(In reply to comment #5) > (From update of attachment 158564 [details]) > Attachment 158564 [details] did not pass mac-ews (mac): > Output: http://queues.webkit.org/results/13502743 The bot is probably sick, this error doesn't make sense if the compile step passed and FrameView::forceLayout is getting linked.
Jocelyn Turcotte
Comment 7 2012-08-16 08:30:57 PDT
Committed r125781: <http://trac.webkit.org/changeset/125781>
Note You need to log in before you can comment on or make changes to this bug.