[Webkit-unassigned] [Bug 132350] AX: [PATCH] procedures for automated atk text and caret tests

[bugzilla-daemon at webkit.org]

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





--- Comment #4 from Mario Sanchez Prada <mario at webkit.org>  2014-05-01 04:02:25 PST ---
(In reply to comment #3)
> Hi Mario
> 
> Thank you for the warning :)
> Now I won't spend too much time on a bug, that will look difficult. But one of 
> the bugs (bug #13941) is already fixed and just for the sake of this bug we 
> could try to patch testatk.c with my auto test method. It would help to keep 
> stability of webkit1 in the maintanance period.

I'm not sure about this, to be honest. It would mean putting effort on testing something in a way that we know will go away, while keeping the main trunk untested, which is where I think the effort should be put instead.

> Another advantage of my patch: it helps much in diagnosing other atk bugs, 
> because new tests can be added so quickly.

Yes, testatk.c is easier to handle than layout tests for this (too ATK specific) kind of tests, but again that's code that will go away and that is no longer in trunk, so I see little point on that approach.

> If you think it would be too much work, then yes, I can prepare an old 
> fashioned test for that.
> 
By old-fashioned test you mean a layout test? If so, then yes I do think that would be better :)

> By the time we make the decision, I will try to gather information about 
> Debian and their plans regarding webkit2.

Cool thanks!
> 
> Jarek

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