Bug 8672 - Red outline from web inspector reappears after inspector is closed
Summary: Red outline from web inspector reappears after inspector is closed
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-04-30 17:07 PDT by bogo_lode
Modified: 2006-06-17 01:51 PDT (History)
4 users (show)

See Also:


Attachments
a screenshot of the overlap. (375.11 KB, image/png)
2006-04-30 17:09 PDT, bogo_lode
no flags Details
Patch (3.18 KB, patch)
2006-06-11 11:50 PDT, mitz
timothy: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description bogo_lode 2006-04-30 17:07:34 PDT
red outline gets drawn when web inspector is closed and new pages load. it also overlaps on other apps when webkit is in the background and the page finishes loading.
Comment 1 bogo_lode 2006-04-30 17:09:36 PDT
Created attachment 8045 [details]
a screenshot of the overlap.
Comment 2 Mark Rowe (bdash) 2006-04-30 17:14:59 PDT
Confirmed with r14125.

To reproduce:
1) Open WebKit.app (or Safari w/WebKit)
2) Right-click, Inspect Element.
3) Close the inspector.
4) Hit refresh, and note that the red outline is drawn around the page after it loads.
Comment 3 David Kilzer (:ddkilzer) 2006-06-11 04:36:58 PDT
(In reply to comment #2)
> 4) Hit refresh, and note that the red outline is drawn around the page after it
> loads.

Also, if you bring other Safari windows or even windows from other application to the front (so that the previously inspected window is now partially or totally hidden), the red outline is drawn over ALL windows!

Makes me wonder what would happen if the window were sent to the dock when the red outline was drawn!
Comment 4 David Kilzer (:ddkilzer) 2006-06-11 05:01:40 PDT
(In reply to comment #3)
> Makes me wonder what would happen if the window were sent to the dock when the
> red outline was drawn!

The red outline is drawn where the window would have been when it's in the dock.
Comment 5 mitz 2006-06-11 11:50:31 PDT
Created attachment 8810 [details]
Patch

This is a fix for the "outline reappears" bug. The problem with the outline being in front of other windows is a separate issue (it is currently in a window level above all other normal windows instead of being a child window).
Comment 6 David Kilzer (:ddkilzer) 2006-06-11 12:57:52 PDT
(In reply to comment #5)
> [...] The problem with the outline
> being in front of other windows is a separate issue (it is currently in a
> window level above all other normal windows instead of being a child window).

Filed Bug 9403 for the above issue.
Comment 7 Timothy Hatcher 2006-06-11 13:02:39 PDT
Comment on attachment 8810 [details]
Patch

r=me
Comment 8 Timothy Hatcher 2006-06-11 13:05:34 PDT
Committed r14812.
Comment 9 Darin Adler 2006-06-11 18:24:46 PDT
It's not right to unconditionally use hostWindow. The client is not obligated to set hostWindow at all. For Safari, there will always be a hostWindow. But in general, the window you're in is either the actual physical window, or if that's nil, the hostWindow.

Setting up an observer that tracks this as the hostWindow value changes and the window possibly changes as well is an exercise left to the coder.
Comment 10 mitz 2006-06-17 01:51:56 PDT
(In reply to comment #9)

Tracking the WebView's window and hostWindow turned out to be complicated and inadequate for framesets. Bug 9479 suggests a WebFrame-based approach.