wxLocalFSHandler::OpenFile() is constructing wxFSFile with incomplete location

[wsu]

The wxFSFile::GetLocation() docs say the returned string includes the protocol. Also, if you look at handlers, e.g. wxMemoryFSHandlerBase and wxArchiveFSHandler, they provide the full location information when creating wxFSFile. However, wxLocalFSHandler::OpenFile() does not create the wxFSFile with protocol information.

This can be seen when running the html/help sample in wxMSW. If you use File | Help to open the help viewer, and then push the Next (wxID_HTML_DOWN) button, nothing happens. However, if you apply the attached patch, the Next button will work (as will the Previous button). The problem is that the wxHtmlBookRecord objects are being passed the wxFSFile::GetLocation() values, but those values are only partial matches for the keys used by wxHtmlHelpWindow::m_PagesHash.

[vadz]

This looks correct to me but OTOH I'm almost sure that the current code was written like this intentionally so I wonder if this could break something else.

Vaclav, would you remember by chance why was it written like this or even just whether this was indeed intentional or not?

[vaclavslavik]

I don't think it was intentional (the example URLs in the docs are something I wouldn't use later than in 2000-or-so when this was originally written... jeez, I'm old). Looks like a right thing to do to me, but there's no telling when side-effect it may have on this convoluted mess.

[vadz]

Thanks for the confirmation. As for breakage, there is only one way to find out: let's apply this and see if anybody complains...

[VZ]

(In [72649]) Do return the protocol part from GetLocaltion() for local wxFSFiles.

wxLocalFSHandler created wxFSFile without the protocol information which means
that calling GetLocaltion() on this file later doesn't return it, contrary to
the documentation.

Do include the protocol to fix this.

Closes #14638.