REGRESSION (r31177): Inspector highlighting of webpage not cleared when going to new URL

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

[Gavin Sherlock]

Steps to reproduce:

1.  Go to any webpage, right click and Inspect Element.
2.  Arrange windows to make Inspector and web page both visible.
3.  Select DOM node, so corresponding part of web page is highlighted
4.  Switch to web page, and go to new url (e.g. clicking on link).

Expected results:

Highlighting on page will clear

Actual Results:

Highlighting on page persists

Regression:  Not present in Safari 3.1, present in r31201.  Tested on 10.5.2 on dual G5

[Gavin Sherlock]

I should also note that you can resize the webpage, and the highlighting will stay where it is, even to the point that it can no longer overlap the page that it was originally highlighting.

[Adam Roben (:aroben)]

I can't reproduce this in ToT on Windows. Maybe it's a Mac-only issue? I think Tim was seeing something similar.

[Timothy Hatcher]

I know what caused this. When the drawing code was moved to WebCore the fade value is ignored in WebNodeHighlightView. We should just remove the fade.

[Timothy Hatcher]

Created attachment 19927 [details]
Patch

 WebKit/mac/ChangeLog                            |   28 ++++++++
 WebKit/mac/WebCoreSupport/WebInspectorClient.mm |   25 +++-----
 WebKit/mac/WebInspector/WebNodeHighlight.h      |   14 +---
 WebKit/mac/WebInspector/WebNodeHighlight.m      |   77 +++--------------------
 WebKit/mac/WebInspector/WebNodeHighlightView.h  |    8 ---
 WebKit/mac/WebInspector/WebNodeHighlightView.m  |   28 ++-------
 6 files changed, 53 insertions(+), 127 deletions(-)

[Adam Roben (:aroben)]

Comment on attachment 19927 [details]
Patch

 11         how just detaches when it is hidden.

Typo: how -> now

r=me

[Timothy Hatcher]

Landed in r31209.