[Webkit-unassigned] [Bug 67582] Calling nativeImageForCurrentFrame() causes assertion failure: m_verifier.isSafeToUse()

[bugzilla-daemon at webkit.org]

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





--- Comment #12 from Brady Eidson <beidson at apple.com>  2012-02-16 10:10:44 PST ---
(In reply to comment #10)
> (In reply to comment #9)
> > > And I also hit this bug in Gtk port when implementing some unit tests for our icon database client
> > 
> > It indicates that there is likely a problem in the code (maybe for multiple platforms), and it needs a volunteer to fix it.
> 
> Well, unless I'm missing something it's pretty clear to me that IconDatabase ::synchronousIconForPageURL cannot be used by any platform as it requires a lock in the mutex used to verify refcounting. This issue was likely never addressed as 1- it does not obviously affect release builds and 2- the code is kind of correct, because even if you don't get the lock, nothing wrong will happen as long as you call it from the same thread that created the image data.

Mac uses it without asserting, and your suggestions as to why it might okay (nothing wrong happening, etc etc) are accurate AFAIK

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.