Summary: | [Windows] Windows is incorrectly using a LayerTypeTiledBackingLayer | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brent Fulgham <bfulgham> | ||||
Component: | Layout and Rendering | Assignee: | Brent Fulgham <bfulgham> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, simon.fraser | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Brent Fulgham
2013-08-13 15:01:13 PDT
Created attachment 208681 [details]
Patch
Comment on attachment 208681 [details]
Patch
r+ but it would be nice to re-enable the tests that should have caught this failure.
rfong and I have a separate task to clean up the various skipped tests on Windows. The tiling tests will get picked up as part of that. Committed r154030: <http://trac.webkit.org/changeset/154030> Comment on attachment 208681 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=208681&action=review > Source/WebCore/platform/graphics/ca/win/PlatformCALayerWin.cpp:119 > + ASSERT((layerType != LayerTypeTiledBackingLayer) && (layerType != LayerTypePageTiledBackingLayer)); These should be two separate assertions. That way you can tell, if the assertion fires, which bad layer type you are dealing with. Comment on attachment 208681 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=208681&action=review > Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp:2689 > +#if PLATFORM(WIN) > + PlatformCALayer::LayerType layerType = useTiledLayer ? PlatformCALayer::LayerTypeWebTiledLayer : PlatformCALayer::LayerTypeWebLayer; > +#else > + PlatformCALayer::LayerType layerType = useTiledLayer ? PlatformCALayer::LayerTypeTiledBackingLayer : PlatformCALayer::LayerTypeWebLayer; > +#endif > + > + m_layer = PlatformCALayer::create(layerType, this); Finding this would have been much easier if we had an ASSERT() when PlatformCALayer::create() fails (or a Log() it happens frequently). |