2011-05-26 David Levin <levin@chromium.org>
authorDavid Levin <levin@chromium.org>
Fri, 27 May 2011 02:16:21 +0000 (02:16 +0000)
committerAdemar de Souza Reis Jr <ademar.reis@openbossa.org>
Tue, 2 Aug 2011 17:50:04 +0000 (14:50 -0300)
commitb5578491e02b4b57346d0b3e6e7e3fbd51a95dec
treef23ddee5fa9d902aa6acd1b5cd7ee69515c2f113
parent02c9701b50dfdc15ada78399f05abdd01392d4ed
2011-05-26  David Levin  <levin@chromium.org>

        Reviewed by Dmitry Titov.

        WebKit's font notification has problems when the WebKit main thread != UI thread.
        https://bugs.webkit.org/show_bug.cgi?id=61391

        This doesn't happen in DumpRenderTree, so it needs a unit test which is taking me
        some time to write correctly. In the meantime, this issues happens to be causing
        some crashes in Chrome so here's the fix alone for the time being.

        * platform/graphics/mac/FontCacheMac.mm:
        (WebCore::invalidateFontCache): Ensure that FontCache::invalidate is only called on WebKit's main thread.
        (WebCore::fontCacheRegisteredFontsChangedNotificationCallback): Call common function for font cache invalidation.
         Note that the call to fontCache() is fine since the singleton is initialized well before calling this function. Theoretically,
         there could be a problem due to a lack of a memory barrier but that is highly unlikely and this is debug only code.
        (WebCore::fontCacheATSNotificationCallback): Ditto.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@87462 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/mac/FontCacheMac.mm