Comment 469 for bug 263435

Revision history for this message
In , 7-postmaster-alishomepage-com (7-postmaster-alishomepage-com) wrote :

Hello

(In reply to comment #456)
> So, most likely a proper way to SetFocus() would be to find browser window's
> HWND by using some built-in method from the browser code itself.

Does that resemble the code in coment 446 ?

> nsCOMPtr<nsIWidget> widget;
> win->GetPluginWidget(getter_AddRefs(widget));
> while (widget != NULL)
> {
> widget->SetFocus(false);
> widget = widget->GetParent();
> }

Roman, can you please test with a Windows patch that does not SetFocus(false) to everything but rather finds the topmost widget (i.e. the widget just before the GetParent became NULL) and call SetFocus(true) on that one?

Cheers

S. Ali Tokmen
http://ali.tokmen.com/