Bug 62512 – [GTK] Remove webkit_web_view_get_selected_text

• Component: WebKitGTK
• Status: RESOLVED FIXED

[Xan Lopez]

It's unused *and* private *and* can what it does can be done with the DOM bindings, so we can just nuke it.

[Xan Lopez]

Created attachment 96881 [details]
Patch

[Andreas Kling]

Comment on attachment 96881 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=96881&action=review

I'm not qualified to make decisions about the Gtk API, but it looks like a reasonable change to me.

Note that the Midori browser currently uses this API, so it would be nice of you to let them know it's being removed. :)

> Source/WebKit/gtk/ChangeLog:8
> +        It's no longer used and is a privated method, so it can be

Typo, private.

[Martin Robinson]

:/ I guess if Midori is using it we should make it deprecated with a note to use the DOM APIs and then remove it in some future release.

[Xan Lopez]

(In reply to comment #3)
> :/ I guess if Midori is using it we should make it deprecated with a note to use the DOM APIs and then remove it in some future release.

Maybe it was not clear in the first comment, but this method is *private*. It's not exported in any public header, so whoever is using it does it at his own risk.

[Martin Robinson]

(In reply to comment #4)
> (In reply to comment #3)
> > :/ I guess if Midori is using it we should make it deprecated with a note to use the DOM APIs and then remove it in some future release.
> 
> Maybe it was not clear in the first comment, but this method is *private*. It's not exported in any public header, so whoever is using it does it at his own risk.

Apologies, I was thrown off momentarily because it had gtkdoc and Midori and claws are using it. I thought perhaps you had made a mistake in thinking it private. I don't see it on the documentation site though, so I believe that you are correct that it is safe to remove. 

I feel that we should not use gtkdoc for private methods. It seems too easy for them slip into the public documentation and sends the wrong message downstream. CCing Christian so that he has some advance warning.

[Xan Lopez]

(In reply to comment #5)
> Apologies, I was thrown off momentarily because it had gtkdoc and Midori and claws are using it. I thought perhaps you had made a mistake in thinking it private. I don't see it on the documentation site though, so I believe that you are correct that it is safe to remove. 

Maybe this is because it was used at some point in DRT (and that's the reason why it's in a header and marked with WEBKIT_API too).

> 
> I feel that we should not use gtkdoc for private methods. It seems too easy for them slip into the public documentation and sends the wrong message downstream. CCing Christian so that he has some advance warning.

Right.

FWIW, we just cannot remove public methods. What you suggested (deprecate + remove later) is what has to happen, it's not really up to debate. With private methods, of course, we can do whatever we want.

[Xan Lopez]

Landed in r88621, closing.