Bug 128490

Summary:

[iOS][wk2] ASSERT(WebThreadIsCurrent()) in [WebDisplayLinkHandler handleDisplayLink:]

Product:

WebKit

Reporter:

Tim Horton <thorton>

Component:

WebCore Misc.

Assignee:

Tim Horton <thorton>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

andersca, commit-queue, dino, sam, simon.fraser

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
patch	none

Description Tim Horton 2014-02-08 23:19:14 PST

[WebDisplayLinkHandler handleDisplayLink:] has an ASSERT(WebThreadIsCurrent()).

We don't use the Web Thread in WebKit2, but it looks to me like isMainThread() does the right thing (main thread if web thread is disabled, web thread otherwise), so let's use that.

Comment 1 Tim Horton 2014-02-08 23:21:47 PST

Created attachment 223613 [details]
patch

Hopefully someone can clarify if isMainThead does the right thing. And maybe we should audit other callers of WebThreadIsCurrent?

Comment 2 WebKit Commit Bot 2014-02-09 16:21:34 PST

Comment on attachment 223613 [details]
patch

Clearing flags on attachment: 223613

Committed r163758: <http://trac.webkit.org/changeset/163758>

Comment 3 WebKit Commit Bot 2014-02-09 16:21:37 PST

All reviewed patches have been landed.  Closing bug.