RESOLVED FIXED 141223
[iOS WK2] Assert in ScrollingTreeOverflowScrollingNodeIOS::updateAfterChildren() on tab switching
https://bugs.webkit.org/show_bug.cgi?id=141223
Summary [iOS WK2] Assert in ScrollingTreeOverflowScrollingNodeIOS::updateAfterChildre...
Simon Fraser (smfr)
Reported 2015-02-03 17:24:00 PST
[iOS WK2] Assert in ScrollingTreeOverflowScrollingNodeIOS::updateAfterChildren() on tab switching
Attachments
Patch (45.22 KB, patch)
2015-02-03 17:30 PST, Simon Fraser (smfr)
no flags
EWS testing (50.57 KB, patch)
2015-02-03 20:15 PST, Simon Fraser (smfr)
no flags
More EWSing (52.84 KB, patch)
2015-02-03 22:57 PST, Simon Fraser (smfr)
simon.fraser: review+
Simon Fraser (smfr)
Comment 1 2015-02-03 17:30:38 PST
Simon Fraser (smfr)
Comment 2 2015-02-03 17:31:24 PST
This patch will almost certainly break on non-Apple platforms. Will fix before submitting.
Simon Fraser (smfr)
Comment 3 2015-02-03 17:31:47 PST
Tim Horton
Comment 4 2015-02-03 18:07:41 PST
Comment on attachment 245995 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=245995&action=review > Source/WebCore/platform/graphics/GraphicsLayer.h:198 > + enum class LayerType { should this just be "Type"? So that its name is WebCore::GraphicsLayer::Type instead of WebCore::GraphicsLayer::LayerType? > Source/WebCore/platform/graphics/GraphicsLayer.h:317 > + no. > Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp:3183 > // move over animations Get rid of this comment?
Simon Fraser (smfr)
Comment 5 2015-02-03 20:15:42 PST
Created attachment 246010 [details] EWS testing
Simon Fraser (smfr)
Comment 6 2015-02-03 22:57:02 PST
Created attachment 246020 [details] More EWSing
Simon Fraser (smfr)
Comment 7 2015-02-04 09:14:12 PST
Comment on attachment 246020 [details] More EWSing Carrying Tim's r+ forward
Simon Fraser (smfr)
Comment 8 2015-02-04 09:15:24 PST
Note You need to log in before you can comment on or make changes to this bug.