Bug 8535 – Incorrect cyan debug mode painting

• Component: Layout and Rendering
• Status: RESOLVED WORKSFORME

[Alexey Proskuryakov]

The attached test case paints incorrectly in debug builds.

[Alexey Proskuryakov]

Created attachment 7910 [details]
test case

[Alexey Proskuryakov]

From [WebFrameView drawRect:]:

#ifndef NDEBUG
        if ([[self _scrollView] drawsBackground]) {
            [[NSColor cyanColor] set];
            NSRectFill(rect);
        }
#endif

Since RenderWidget::paint() calls paintBoxDecorations() first, and m_widget->paint() later, this paints over an already painted background.

Removing this block has fixed the problem in Safari, but DumpRenderTree didn't paint the IFRAME background at all then. I am not sure how much related these problems are, though.

[Alexey Proskuryakov]

*** Bug 9982 has been marked as a duplicate of this bug. ***

[David Kilzer (:ddkilzer)]

It's really interesting to see what gets repainted when various actions are taken, though!

[Alexey Proskuryakov]

*** Bug 11799 has been marked as a duplicate of this bug. ***

[Robert Hogan]

This works fine for me on a debug build of DumpRenderTree for the Qt port and a debug build of QtTestBrowser. OK to close?

[Alexey Proskuryakov]

This code is Mac specific, so testing Qt is not necessarily relevant. However, I cannot reproduce on Mac either, and haven't seen such cyan painting in years.