Bug 4491 - DOM L2 Events tests using addEventListener broken
Summary: DOM L2 Events tests using addEventListener broken
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Darin Adler
URL:
Keywords:
: 4492 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-08-17 08:49 PDT by Curt Arnold
Modified: 2005-08-17 14:12 PDT (History)
1 user (show)

See Also:

Attachments
Patch for l2 events tests (5.55 KB, patch)
2005-08-17 08:57 PDT, Curt Arnold 		darin: review+
Details | Formatted Diff | Diff
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 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. ***