[webkit-reviews] review requested: [Bug 50083] [Gtk] ASSERT(d->m_response.isNull()) in contentSniffedCallback : [Attachment 74934] Fix for the bug

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Nov 26 06:51:45 PST 2010


Sergio Villar Senin <svillar at igalia.com> has asked  for review:
Bug 50083: [Gtk] ASSERT(d->m_response.isNull()) in contentSniffedCallback
https://bugs.webkit.org/show_bug.cgi?id=50083

Attachment 74934: Fix for the bug
https://bugs.webkit.org/attachment.cgi?id=74934&action=review

------- Additional Comments from Sergio Villar Senin <svillar at igalia.com>
Final version of the patch:
   * connect to content-sniffed signal if handle->shouldContentSniff()
   * fixed an potential early finalization of an object
   * added comments with the "proper" fix inside the cache code to be migrated
to libsoup


More information about the webkit-reviews mailing list