Bug 4491

Summary:

DOM L2 Events tests using addEventListener broken

Product:

WebKit

Reporter:

Curt Arnold <curt.arnold>

Component:

DOM

Assignee:

Darin Adler <darin>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

darin

Priority:

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
Patch for l2 events tests	darin: review+

Description Curt Arnold 2005-08-17 08:49:47 PDT

The initial load the the DOM L2 Events tests in WebCore/layout-tests/html/level2/events were from a stale 
snapshot that did not include the changes resulting from the fix to http://www.w3.org/Bugs/Public/
show_bug.cgi?id=1137.  The major observable change was that the second parameter to addEventListener 
should have been monitor.handleEvent and not monitor.

Comment 1 Curt Arnold 2005-08-17 08:57:34 PDT

Created attachment 3437 [details]
Patch for l2 events tests

The whitespace in the rendered tests have also been changed since the snapshot.
 This patch ignores the whitespace changes within the test scripts.  The patch
does not contain changes to the -expected files that may result from the test
changes.  Before committing, the -expected.txt's for the modified tests should
be deleted and run-webkit-tests run twice (once to produce the -expected.txt
and once to make sure they don't change).

Comment 2 Darin Adler 2005-08-17 11:24:14 PDT

*** Bug 4492 has been marked as a duplicate of this bug. ***