Bug 14682

Summary: REGRESSION (24285-24399): alerts do not appear at W3C RTL test page
Product: WebKit Reporter: Alexey Proskuryakov <ap>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: sam, zwarich
Priority: P1 Keywords: InRadar, Regression
Version: 523.x (Safari 3)   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.w3.org/International/tests/test-rtl-chrome-1
Attachments:
Description Flags
reduction
none
patch darin: review+

Description Alexey Proskuryakov 2007-07-19 21:42:30 PDT
Steps to reproduce:
1) Open the bug URL
2) Click any of the "Click Me" buttons.

Resuls: an alert doesn't appear; there's an error in JS Console:

Object  (result of expression alert) does not allow calls.
http://www.w3.org/International/tests/test-rtl-chrome-1

Tested with nightly r24399.
Comment 1 David Kilzer (:ddkilzer) 2007-07-20 04:29:29 PDT
Works on Safari 2.0.4 (419.3) with original WebKit on Mac OS X 10.4.10 (8R218).

Comment 2 David Kilzer (:ddkilzer) 2007-07-20 04:30:54 PDT
<rdar://problem/5349746>
Comment 3 Alexey Proskuryakov 2007-07-22 13:23:12 PDT
Created attachment 15626 [details]
reduction
Comment 4 Alexey Proskuryakov 2007-07-22 13:40:19 PDT
Two commits that look somewhat suspicious are <http://trac.webkit.org/projects/webkit/changeset/24350> and <http://trac.webkit.org/projects/webkit/changeset/24287>.
Comment 5 mitz 2007-07-22 14:49:26 PDT
Probably r24350: fake getElementById properties now get priority over the autogenerated properties of the window prototype (see Window::getOwnPropertySlot)
Comment 6 Cameron Zwarich (cpst) 2007-07-22 19:22:20 PDT
(In reply to comment #4)
> Two commits that look somewhat suspicious are
> <http://trac.webkit.org/projects/webkit/changeset/24350> and
> <http://trac.webkit.org/projects/webkit/changeset/24287>.

I tried r24287 and the test passes, so it must be a later revision.
Comment 7 Sam Weinig 2007-07-22 22:17:50 PDT
Created attachment 15642 [details]
patch
Comment 8 Darin Adler 2007-07-22 22:22:36 PDT
Comment on attachment 15642 [details]
patch

r=me
Comment 9 Alexey Proskuryakov 2007-07-22 22:37:15 PDT
There's a mismatched closing tag here:

+    <div id="alert"></button>

Comment 10 Sam Weinig 2007-07-22 22:39:45 PDT
(In reply to comment #9)
> There's a mismatched closing tag here:
> 
> +    <div id="alert"></button>
> 

Eek, you're right.  I will fix it.
Comment 11 Sam Weinig 2007-07-22 22:43:36 PDT
Landed in r24528.