Bug 6643 – REGRESSION: Tree view repaints lines without erasing them first

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

Summary: each time you switch Safari to the foreground with the web inspector open, all the lines above 
the selected element in the tree view are painted over themselves. After a few switches, they become 
brighter and aliased (with darker, sharp shadows).

To reproduce: open the web inspector, select an element in the middle of the tree view, then switch Safari 
to the background and back to the foreground several times (5 are enough to make the bug noticeable).

I just noticed what may be a related problem: when you have an element selected in the tree view and you 
click to select another element, all the elements in between are "double-painted" momentarily as the 
selection changes.

Created attachment 5756 [details]
Screenshot

The problem has worsened lately. You can now reproduce the double-painting simply by changing the selection in the tree view several times (without making it scroll). It is also noticeable in the highlight.

The latest regression happened in the r19843-r19850 range.

[Timothy Hatcher]

I am in the process of converting the inspector tree to all HTML, so this will go away soon. Still might be good to fix.

[David Kilzer (:ddkilzer)]

Regressions are P1. :(

[Mark Rowe (bdash)]

<rdar://problem/5061017>

(In reply to comment #3)
> The latest regression happened in the r19843-r19850 range.

r19848 (parts landed in r19849 and r19850).

[Timothy Hatcher]

This will be fixed with my changes to make the tree be pure HTML/CSS/JavaScript.

http://trac.webkit.org/projects/webkit/browser/branches/WebInspectorRefresh

I think the main cause of this double paint was the fact that the WebInspectorOutlineView subclass didn't pain an opaque background, and didn't clear before drawing.

[Timothy Hatcher]

This was fixed on the inspector branch and merged to trunk in r20986.