[webkit-reviews] review granted: [Bug 99265] [EFL][WK2] Display page favicons in MiniBrowser : [Attachment 168769] Patch

[bugzilla-daemon at webkit.org]

Gyuyoung Kim <gyuyoung.kim at samsung.com> has granted Christophe Dumez
<christophe.dumez at intel.com>'s request for review:
Bug 99265: [EFL][WK2] Display page favicons in MiniBrowser
https://bugs.webkit.org/show_bug.cgi?id=99265

Attachment 168769: Patch
https://bugs.webkit.org/attachment.cgi?id=168769&action=review

------- Additional Comments from Gyuyoung Kim <gyuyoung.kim at samsung.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=168769&action=review


> Tools/MiniBrowser/efl/main.c:345
> +	   evas_object_ref(icon);

You remove icon object without doing unref at 334 line. So, do you need to
increase ref count here ?