1. Go to http://build.webkit.org/LeaksViewer/
2. Open the Inspector's debugger and put a breakpoint in LeaksViewer.urlPromptButtonClicked in LeaksViewer.js
3. Click back on the main page
4. Click on the "Fetch leaks" button
The breakpoint is hit in the debugger, but the Inspector doesn't come to the front. In WebKit1, it does come to the front.
I think the bug is caused by WebInspectorFrontendClient::bringToFront being unimplemented.
If we implement this, and remove the code that immediately shows the Inspector window when it's first created, then the Inspector window might stop showing up when running run-webkit-tests.
The bringToFront function is only used after the inspector has a window created by the client. The client is required to show it the first time. So we can't remove the code to immediately show it, unless we refactor things.
Created attachment 90353 [details]
Attachment 90353 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit..." exit_code: 1
Source/WebKit2/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] 
Total errors found: 1 in 11 files
If any of these errors are false positives, please file a bug against check-webkit-style.
(In reply to comment #4)
> The bringToFront function is only used after the inspector has a window created by the client. The client is required to show it the first time. So we can't remove the code to immediately show it, unless we refactor things.
I thought WebKit1 didn't immediately show it, but maybe I'm wrong.
Comment on attachment 90353 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=90353&action=review
>> +2011-04-20 Timothy Hatcher <firstname.lastname@example.org>
> ChangeLog entry has no bug number [changelog/bugnumber] 
I think check-webkit-style doesn't know about https: for short bug URLs. You should file a bug about that.
> + // FIXME: support bring to front in docked mode here.
You could put this FIXME in the Windows code, too.
> +void WebInspectorProxy::platformBringToFront()
> + if (!m_inspectorWindow)
> + return;
> + ASSERT(::IsWindow(m_inspectorWindow));
> + ::ShowWindow(m_inspectorWindow, SW_SHOW);
I don't think this will be sufficient on Windows. I think this would do the trick instead:
::SetWindowPos(m_inspectorWindow, HWND_TOP, 0, 0, 0, 0, SWP_SHOWWINDOW | SWP_NOMOVE | SWP_NOSIZE);
I file a bug about the style bot a while ago.
Landed in r84424 with Adam's feedback applied.