Bug 101727 – [GTK] Isolate the GTK/Gail/Pango specific code in accessibility/gtk

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit EFL

[Mario Sanchez Prada]

This is a bug to track work initially started in bug 99578, to provide conditional compilation units for isolating the GTK specific code in accessibility/gtk, so most of the code in there can be later reused in other ports, such as EFL

[Mario Sanchez Prada]

Created attachment 173245 [details]
Patch proposal

Here comes the patch, properly filling the ChangeLog and addressing the review comments from bug 99578.

[Mario Sanchez Prada]

Committed r134094: <http://trac.webkit.org/changeset/134094>

[Martin Robinson]

I think we should just replace this code for both platforms with something platform-independent, if possible.

[Alejandro Piñeiro]

(In reply to comment #3)
> I think we should just replace this code for both platforms with something platform-independent, if possible.

This was discussed at bug 99578 comment 4, 5 and 6. The idea is replace the gail/pango code used to implement some methods of AtkText. Mario suggested stuff like TextIterator, that it is already in use. But that task can be long, so first step is isolate that code (this patch) while working on the final and platform-independent solution. In summary: this isolation is a temporary solution.

[Martin Robinson]

(In reply to comment #4)
> (In reply to comment #3)
> > I think we should just replace this code for both platforms with something platform-independent, if possible.
> 
> This was discussed at bug 99578 comment 4, 5 and 6. The idea is replace the gail/pango code used to implement some methods of AtkText. Mario suggested stuff like TextIterator, that it is already in use. But that task can be long, so first step is isolate that code (this patch) while working on the final and platform-independent solution. In summary: this isolation is a temporary solution.

Hrm. Do you mean that there will be many patches and the work will be incomplete during that time?

[Mario Sanchez Prada]

(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > I think we should just replace this code for both platforms with something platform-independent, if possible.
> > 
> > This was discussed at bug 99578 comment 4, 5 and 6. The idea is replace the gail/pango code used to implement some methods of AtkText. Mario suggested stuff like TextIterator, that it is already in use. But that task can be long, so first step is isolate that code (this patch) while working on the final and platform-independent solution. In summary: this isolation is a temporary solution.
> 
> Hrm. Do you mean that there will be many patches and the work will be incomplete during that time?

As for WebKitGTK the work won't ever be incomplete (it will for other ports, though), since it will keep working as it used to do so far. These conditional compilation units were added just to ease the work of other ports (EFL) should they want to reuse the non-GTK dependant ATK code (which is 99% already).

However, I'd expect that over time (probably while implementing those notimplemented() bits in the #if !PLATFORM(GTK) regions) most of those conditional bits (e.g. pango / gail) will be implemented in a more generic way so they could be shared by all the ports. Which I think is what you suggested in your first comment.