Bug 132034 - ASSERTION FAILED: scrollerImp == scrollbarPainterForScrollbar(_scrollbar) on two API tests
Summary: ASSERTION FAILED: scrollerImp == scrollbarPainterForScrollbar(_scrollbar) on ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tim Horton
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2014-04-22 18:33 PDT by Tim Horton
Modified: 2014-04-22 18:38 PDT (History)
3 users (show)

See Also:


Attachments
patch (1.76 KB, patch)
2014-04-22 18:34 PDT, Tim Horton
simon.fraser: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tim Horton 2014-04-22 18:33:46 PDT
scrollerImpWithStyle:controlSize:horizontal:replacingScrollerImp: moves the delegate
from the old scrollerImp to the new one, and also happens to call shouldUseLayerPerPartForScrollerImp.
Since scrollerImpWithStyle: has not returned yet, the scrollbarPainterForScrollbar still returns the old scrollerImp,
so this assertion fires. It's safe to remove this because supportsUpdateOnSecondaryThread doesn't make use of the imp.

<rdar://problem/16624332>
Comment 1 Tim Horton 2014-04-22 18:34:50 PDT
Created attachment 229937 [details]
patch
Comment 2 Tim Horton 2014-04-22 18:38:20 PDT
http://trac.webkit.org/changeset/167693