[webkit-reviews] review granted: [Bug 173408] [GTK] Do not use the C API in WebGtkExtensionManager : [Attachment 312969] Patch

[bugzilla-daemon at webkit.org]

Michael Catanzaro <mcatanzaro at igalia.com> has granted Carlos Garcia Campos
<cgarcia at igalia.com>'s request for review:
Bug 173408: [GTK] Do not use the C API in WebGtkExtensionManager
https://bugs.webkit.org/show_bug.cgi?id=173408

Attachment 312969: Patch

https://bugs.webkit.org/attachment.cgi?id=312969&action=review




--- Comment #3 from Michael Catanzaro <mcatanzaro at igalia.com> ---
Comment on attachment 312969
  --> https://bugs.webkit.org/attachment.cgi?id=312969
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=312969&action=review

> Source/WebKit2/WebProcess/gtk/WebGtkExtensionManager.cpp:-56
> -    ASSERT(userData);

Why did you remove this line? I would keep it.