[Webkit-unassigned] [Bug 89810] [GTK] Test /webkit2/WebKitFindController/hide always fails in Xvfb
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Apr 18 09:27:22 PDT 2013
https://bugs.webkit.org/show_bug.cgi?id=89810
Carlos Garcia Campos <cgarcia at igalia.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #198714|review? |review-
Flag| |
--- Comment #4 from Carlos Garcia Campos <cgarcia at igalia.com> 2013-04-18 09:25:40 PST ---
(From update of attachment 198714)
View in context: https://bugs.webkit.org/attachment.cgi?id=198714&action=review
Much better! Just some minor issues and a memory leak.
> Source/WebKit2/UIProcess/API/gtk/tests/WebViewTest.cpp:437
> + m_surface = 0;
This leaks the surface, this is not a smart pointer. You should check if the poiner is valid can call cairo_surface_destroy.
> Source/WebKit2/UIProcess/API/gtk/tests/WebViewTest.cpp:440
> + return cairo_surface_reference(m_surface);
If you are keeping a reference, it's better to return your reference and let the caller decide whether she needs to hold a reference or not.
> Source/WebKit2/UIProcess/API/gtk/tests/WebViewTest.cpp:443
> +gboolean WebViewTest::cairoSurfacesEqual(cairo_surface_t* s1, cairo_surface_t* s2)
Use bool instead of gboolean. This has nothing to do with WebView, you could add it as a static method to Test, as I proposed before.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list