Summary: | Inspector doesn't come to the front when a breakpoint is hit in the debugger in WebKit2 | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Roben (:aroben) <aroben> | ||||
Component: | Web Inspector (Deprecated) | Assignee: | Timothy Hatcher <timothy> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bweinstein, timothy, webkit.review.bot | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Adam Roben (:aroben)
2011-03-30 11:20:05 PDT
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]
Proposed Change
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] [5]
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] Proposed Change View in context: https://bugs.webkit.org/attachment.cgi?id=90353&action=review >> Source/WebKit2/ChangeLog:1 >> +2011-04-20 Timothy Hatcher <timothy@apple.com> > > ChangeLog entry has no bug number [changelog/bugnumber] [5] I think check-webkit-style doesn't know about https: for short bug URLs. You should file a bug about that. > Source/WebKit2/UIProcess/mac/WebInspectorProxyMac.mm:139 > + // FIXME: support bring to front in docked mode here. You could put this FIXME in the Windows code, too. > Source/WebKit2/UIProcess/win/WebInspectorProxyWin.cpp:197 > +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); |