Bug 90919

Summary:

Assertion ASSERTION FAILED: enclosingIntRect(rendererMappedResult) == enclosingIntRect(FloatQuad(result).boundingBox()) when compositing in paginated mode

Product:

WebKit

Reporter:

Simon Fraser (smfr) <simon.fraser>

Component:

Layout and Rendering

Assignee:

Simon Fraser (smfr) <simon.fraser>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

dglazkov, eric, gustavo, koivisto, simon.fraser, webkit.review.bot, xan.lopez

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	koivisto: review+

Simon Fraser (smfr)

Reported 2012-07-10 16:54:42 PDT

ASSERTION FAILED: enclosingIntRect(rendererMappedResult) == enclosingIntRect(FloatQuad(result).boundingBox()) Source/WebCore/rendering/RenderGeometryMap.cpp(85) : WebCore::FloatRect WebCore::RenderGeometryMap::absoluteRect(const WebCore::FloatRect &) const 1 0x409325a WebCore::RenderLayerCompositor::computeCompositingRequirements(WebCore::RenderLayer*, WebCore::RenderLayer*, WebCore::RenderLayerCompositor::OverlapMap*, WebCore::CompositingState&, bool&, bool&) 2 0x40936a5 WebCore::RenderLayerCompositor::computeCompositingRequirements(WebCore::RenderLayer*, WebCore::RenderLayer*, WebCore::RenderLayerCompositor::OverlapMap*, WebCore::CompositingState&, bool&, bool&) 3 0x40937c6 WebCore::RenderLayerCompositor::computeCompositingRequirements(WebCore::RenderLayer*, WebCore::RenderLayer*, WebCore::RenderLayerCompositor::OverlapMap*, WebCore::CompositingState&, bool&, bool&) 4 0x4092983 WebCore::RenderLayerCompositor::updateCompositingLayers(WebCore::CompositingUpdateType, WebCore::RenderLayer*) 5 0x3266312 WebCore::FrameView::updateCompositingLayersAfterLayout() 6 0x3268965 WebCore::FrameView::layout(bool)

Attachments
Patch (5.70 KB, patch) 2012-07-10 18:20 PDT, Simon Fraser (smfr)	no flags	Details Formatted Diff Diff
Patch (5.71 KB, patch) 2012-07-10 18:34 PDT, Simon Fraser (smfr)	koivisto: review+	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Simon Fraser (smfr)

Comment 1 2012-07-10 16:58:32 PDT

The old code would insert a step with isNonUniform=true when pushing the step from the RenderView to the <html>'s RenderBlock. The new code relies on layer->canUseConvertToLayerCoords(), which checks *this* layer for columns, not the layer of the container.

Simon Fraser (smfr)

Comment 2 2012-07-10 18:20:41 PDT

Created attachment 151568 [details] Patch

Gyuyoung Kim

Comment 3 2012-07-10 18:24:35 PDT

Comment on attachment 151568 [details] Patch Attachment 151568 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/13197009

Early Warning System Bot

Comment 4 2012-07-10 18:26:28 PDT

Comment on attachment 151568 [details] Patch Attachment 151568 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/13212006

Early Warning System Bot

Comment 5 2012-07-10 18:26:42 PDT

Comment on attachment 151568 [details] Patch Attachment 151568 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/13199149