[Webkit-unassigned] [Bug 34449] [Gtk] Evaluate and create tests for all the AtkRole's implemented by WebKitGtk

Tue Feb 2 12:59:47 PST 2010

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

Xan Lopez <xan.lopez at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #47899|review?                     |review-
               Flag|                            |

--- Comment #14 from Xan Lopez <xan.lopez at gmail.com>  2010-02-02 12:59:47 PST ---
(From update of attachment 47899)
First of all, thanks a lot for working of this. No doubt we should also work in
improving our DRT coverage, but I think there's no reason at all to not have
our own ATK unit tests, especially if someone has gone through the effort
already of writing them.

I just have a couple of general comments that I believe apply to all your
patches:

- You should use g_test_add, which allows you to pass a couple of functions to
setup/finalize things. This should allow you to reuse a bunch of repeated code
you have all over to place. As an example you can check how it's done in
testloading.c

- Do not use a timeout to bail out of the test. You can add queue an idle, run
the mainloop, and in the idle callback do your stuff and then quit. Otherwise
the 100ms keeps adding up and the tests would take a lot longer to run than
needed.

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