2009-06-17 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Jun 2009 18:22:31 +0000 (18:22 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Jun 2009 18:22:31 +0000 (18:22 +0000)
commite7787dc935fdf1cdf9139ace08ccf17f591861ce
tree9ab0cd53ed1c609e23ba0652a4bd9d8e95479b37
parent4562137af13db0c7d8a3284132ccfc9601a5af74
2009-06-17  Ojan Vafai  <ojan@chromium.org>

        Reviewed by NOBODY (OOPS!).

        If loading a font fails because of the sandbox, we ask the browser process to
        try to load it by calling ensureFontLoaded. If it still fails after
        ensureFontLoaded, we hit a ASSERT_NOT_REACHED.

        This case happens once in a while during browser shutdown. The browser will
        queue a message to the renderer to shutdown, and will then stop answering sync
        messages from the renderer. If the renderer is still loading a page during this
        time, it might try to call the browser process to ask to load a font. The
        browser process will ignore the request, and the font will fail to load, even
        after the second try.

        This is unfortunate, but there is no real risk here, since the renderer will be
        going away as soon as it processes another message.

        This can't be layout tested as it depends on the sandbox.

        https://bugs.webkit.org/show_bug.cgi?id=26484

        * platform/graphics/chromium/FontCacheChromiumWin.cpp:
        (WebCore::fontContainsCharacter):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@44770 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog
WebCore/platform/graphics/chromium/FontCacheChromiumWin.cpp