Bug 22262

Summary: Clicking close ("X") button in docked Web Inspector clears Web Inspector but doesn't close it
Product: WebKit Reporter: Adam Roben (:aroben) <aroben>
Component: Web Inspector (Deprecated)Assignee: Adam Roben (:aroben) <aroben>
Status: RESOLVED FIXED    
Severity: Normal CC: timothy
Priority: P2 Keywords: InRadar, PlatformOnly
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
Attachments:
Description Flags
Patch + ChangeLog darin: review+

Description Adam Roben (:aroben) 2008-11-14 07:45:05 PST 
To reproduce:

1. Go to any page (e.g., <http://webkit.org/>)
2. Open the Inspector
3. Dock the Inspector into the WebView
4. Click the close ("X") button in the upper-left of the Inspector

The Inspector's contents get cleared (i.e., the DOM tree disappears), but the Inspector doesn't close.

Note that bug 14270 introduced the close button in order to give a way to close the docked Inspector, but clearly not everything needed for that bug fix is implemented yet on Windows.
Comment 1 Adam Roben (:aroben) 2008-11-14 07:45:39 PST 
<rdar://problem/6371873>
Comment 2 Adam Roben (:aroben) 2009-01-06 17:38:46 PST 
Created attachment 26482 [details]
Patch + ChangeLog
Comment 3 Darin Adler 2009-01-06 21:53:15 PST 
Comment on attachment 26482 [details]
Patch + ChangeLog

r=me

Hard for me to swallow those C style casts thought.
Comment 4 Adam Roben (:aroben) 2009-01-07 07:40:53 PST 
(In reply to comment #3)
> (From update of attachment 26482 [details] [review])
> r=me
> 
> Hard for me to swallow those C style casts thought.

OK, I'll change them to reinterpret_casts (and simplify them a bit) before landing.

Thanks for the review!
Comment 5 Adam Roben (:aroben) 2009-01-07 07:58:41 PST 
Committed in r39682 <http://trac.webkit.org/changeset/39682>