Bug 65487 - window.open behaves differently in onchange vs onclick event handlers
Summary: window.open behaves differently in onchange vs onclick event handlers
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac (Intel) OS X 10.6
: P2 Minor
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-01 14:35 PDT by Brad Vogel
Modified: 2011-08-01 14:49 PDT (History)
3 users (show)

See Also:

Attachments
Test case (263 bytes, text/html)
2011-08-01 14:38 PDT, Brad Vogel 		no flags Details
Test case (ignore previous) (282 bytes, text/html)
2011-08-01 14:46 PDT, Brad Vogel 		no flags Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Brad Vogel 2011-08-01 14:35:52 PDT 
Repro:
1) Open attached test.html in Chrome 12
2) Click "Open window google.com" button. A new tab will open.
3) Now click the selector and select "google.com".

In step #3, the site will open as a new window (NOT a tab like step #2) and is vulnerable to the browser's popup blocker.

Why does calling window.open with the same parameters in an onclick vs onchange behave differently?
Comment 1 Levi Weintraub 2011-08-01 14:38:00 PDT 
Please attach the test case ;)
Comment 2 Brad Vogel 2011-08-01 14:38:23 PDT 
Created attachment 102559 [details]
Test case
Comment 3 Brad Vogel 2011-08-01 14:38:55 PDT 
(In reply to comment #1)
> Please attach the test case ;)

Attached :).
Comment 4 Levi Weintraub 2011-08-01 14:44:19 PDT 
(In reply to comment #3)
> (In reply to comment #1)
> > Please attach the test case ;)
> 
> Attached :).

Is this the right file? It doesn't seem to correspond with your repro instructions.
Comment 5 Brad Vogel 2011-08-01 14:46:24 PDT 
Created attachment 102560 [details]
Test case (ignore previous)
Comment 6 Levi Weintraub 2011-08-01 14:49:30 PDT 
This is a Chrome issue. Brad'll refile on crbug.com.

Sorry for the extra bs, Brad!