Summary: | [GTK][WPE] Assertion failure in TimerBase inside WebCore::IconRecord::setImageData | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||
Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | beidson, berto, bugs-noreply, buildbot, cdumez, cgarcia, cturner, ggaren, gustavo, mcatanzaro, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | Safari Technology Preview | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Ryosuke Niwa
2017-06-26 23:17:41 PDT
This seems like a very recent regression. I don't recall having this issue a week ago. Hm... I still reproduce this at r218500 so maybe it's not a very recent regression. I'm pretty certain it's not an old assertion though because I do recall being able to browser the Web with debug builds without hitting this before. *** Bug 173524 has been marked as a duplicate of this bug. *** (In reply to Ryosuke Niwa from comment #3) > Hm... I still reproduce this at r218500 so maybe it's not a very recent > regression. I'm hitting this too. I don't remember seeing it until recently. (In reply to Ryosuke Niwa from comment #3) > Hm... I still reproduce this at r218500 so maybe it's not a very recent > regression. I'm pretty certain it's not an old assertion though because I do > recall being able to browser the Web with debug builds without hitting this > before. Ryosuke, I'm curious under what configuration you're seeing this, because IconDatabase no longer exists in trunk WebKit for Mac/iOS. Therefore... you shouldn't be seeing this with Debug Safari. This bug is one month old, so I bet it doesn't affect Mac anymore. It's still causing tons of crashes for WebKitGTK+, though. (In reply to Michael Catanzaro from comment #8) > This bug is one month old, so I bet it doesn't affect Mac anymore. It's > still causing tons of crashes for WebKitGTK+, though. Oh yah whoops, I misread the month Ryosuke posted it. Michael, are you sure you are still getting crashes? WebIconDatabase no longer exists, and I fixed the crash I could reproduce in r219861. If you are still getting crashes with current trunk I need a new backtrace. Created attachment 317788 [details]
Patch
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API Committed r220580: <http://trac.webkit.org/changeset/220580> |