Bug 67031 - [chromium] ASSERTION FAILED: fontCache()->generation() == m_generation
Summary: [chromium] ASSERTION FAILED: fontCache()->generation() == m_generation
Status: RESOLVED DUPLICATE of bug 59552
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-26 04:10 PDT by Tony Gentilcore
Modified: 2011-11-07 02:08 PST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tony Gentilcore 2011-08-26 04:10:52 PDT
Jamesr noticed an ASSERT() fail on the mac bot:
ASSERTION FAILED: fontCache()->generation() == m_generation
 
build log:
http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.6%20(CG)(dbg)/builds/202/steps/webkit_tests/logs/stdio

test was:
fast/css-generated-content/table-before-after-child-add.html

This ASSERT was added by mitz back in http://trac.webkit.org/changeset/35025 , so it is likely a problem w/ the chromium mac port rather than with the ASSERT itself. If this ASSERT is reached, it means that FontCache::invalidate() was called. One place we call that is in WebKit/chromium/src/WebFontCache.cpp::clear(). That gets called on low memory notification (from ChromeRenderProcessObserver::OnPurgeMemory()), and from the ASSERT it seems like that isn't safe to do. That was added way back in http://codereview.chromium.org/267021 .
Comment 1 Peter Kasting 2011-08-26 10:28:36 PDT
At the time I wrote that code, the purge memory call could never happen except manually (it explicitly did not happen in response to memory pressure notifications from the OS).  Perhaps since then someone has hooked this up to another signal.

I don't have a clue about the semantics of this stuff.  It sounds as if invalidate() is never safe to call, which sounds strange.  Feel free to fix however is correct.
Comment 2 Julien Chaffraix 2011-10-21 16:56:34 PDT
FYI, the snow-leopard bots are randomly hitting this ASSERT as part of storage/domstorage/events/basic-body-attribute.html. Look for the test name in http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.6%20(dbg)/builds/5874/steps/webkit_tests/logs/stdio
Comment 3 Yuta Kitamura 2011-11-07 02:08:19 PST

*** This bug has been marked as a duplicate of bug 59552 ***