Bug 74277

Summary: Search with Google in docked Web Inspector should not replace the inspector itself
Product: WebKit Reporter: kevinn <knavia>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, ap, burg, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, sullivan, vsevik, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.7   
Attachments:
Description Flags
Screenshot of google results in docked window. none

kevinn
Reported 2011-12-12 02:26:49 PST
Created attachment 118756 [details] Screenshot of google results in docked window. Within Safari, I'm having trouble running embedded videos from YouTube. Checking web inspector's console I saw a javascript problem, highlighted the term and searched google for it. web inspector would open content in the docked window, which should open in a new tab.
Attachments
Screenshot of google results in docked window. (124.94 KB, image/png)
2011-12-12 02:26 PST, kevinn 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2011-12-12 10:35:12 PST
Steps to reproduce: 1. Open any Web page. 2. Bring up Web Inspector in either docked or separate window mode. 3. Select any text in JS console (type it if necessary). 4. Perform Search with Google using context menu. Results: Web Inspector is replaced with a Google page. Expected results: Not sure. Maybe it should open a new tab/window, or maybe the menu item should not be available. Note: I didn't check whether system-wide Search with Google service has the same issue (it's disabled by default). It's unclear to me if this is a WebKit or a Safari issue.
Vsevolod Vlasov
Comment 2 2011-12-12 13:44:01 PST
This does not happen in Chromium.
kevinn
Comment 3 2011-12-12 14:38:34 PST
(In reply to comment #2) Just to add, in Chrome: - "Search in Google" in web inspector opens a new tab. - Has the context menu item in js console and computed styles and is not available in source pane.
Timothy Hatcher
Comment 4 2011-12-13 13:26:29 PST
I'll take this.
Note You need to log in before you can comment on or make changes to this bug.