Override of wxLogWindow::OnFrameCreate gets never called

[MathiasDz]

Overriding wxLogWindow::OnFrameCreate is useless, because the method gets never called.

wxLogWindow::OnFrameCreate is indirectly called (via wxLogFrame::wxLogFrame) in the wxLogWindow constructor. This calls always wxLogWindow::OnFrameCreate and never OnFrameCreate in the derived class.

A look at 2.9.1 source shows the same problem.

See also:
 http://www.parashift.com/c++-faq-lite/strange-inheritance.html
[23.5] When my base class's constructor calls a virtual function on its this object, why doesn't my derived class's override of that virtual function get invoked?

[vadz]

I think we should just remove this method, it doesn't seem very useful (you could just as well do whatever extra initialization you need after creating wxLogWindow) and making it work would require introducing 2 step creation for wxLogWindow which I'd rather avoid.

Any objections to removing it?

[VZ]

(In [73622]) Remove wxLogWindow::OnFrameCreate() virtual method.

This method couldn't be overridden by the classes deriving from wxLogWindow
because it was called (indirectly) from wxLogWindow ctor itself and so was
completely useless. Just remove it to avoid confusion.

Closes #12763.