296849 - [BEOS] nsWindow::SetForegroundColor calls nsBaseWidget::SetBackgroundColor

Status: VERIFIED DUPLICATE of bug 298171

(Core Graveyard :: Widget: BeOS, defect)

Description

[tqh]

User-Agent:       Mozilla/5.0 (BeOS; U; BeOS BePC; en-US; rv:1.8b2) Gecko/20050604 Firefox/1.0+
Build Identifier: 

nsWindow::SetForegroundColor calls nsBaseWidget::SetBackgroundColor while it
should call SetForegroundColor.

Reproducible: Always

Comment 1

[tqh]

Attachment: calls SetForegroundColor instead

r?

Comment 2

[Sergei Dolgov]

Looks obvious, but i should check first result of those calls in GFX - is it
SetViewColor or SetHightColor or SetLowColor, and what it should be actually:)

Comment 3

[Sergei Dolgov]

Comment on attachment 185480 [details] [diff] [review]
calls SetForegroundColor instead

r=sergei_d
Searched through LXR.
It seems there is now use for those color set methods outside widget-internal
"doing nothing".
or like that
http://lxr.mozilla.org/seamonkey/source/widget/src/mac/nsWindow.cpp#358
which is also "internal".

Comment 4

[tqh]

Moved this code into bug 298171 as I wanted to change viewcolor-handling to
improve scrolling and reduce the need to apply different patches.

*** This bug has been marked as a duplicate of 298171 ***