Bug 24577

Summary:

Fix layout test for v8: LayoutTests/fast/events/onload-single-line-comment.html

Product:

WebKit

Reporter:

Mike Belshe <mbelshe>

Component:

WebCore JavaScript

Assignee:

Mike Belshe <mbelshe>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

All

Attachments:

Description	Flags
patch	fishd: review+

Description Mike Belshe 2009-03-13 10:23:06 PDT

Event handlers with trailing comments fail.

Comment 1 Mike Belshe 2009-03-13 17:10:52 PDT

Created attachment 28615 [details]
patch

Comment 2 Mike Belshe 2009-03-13 17:13:48 PDT

In reviewing my patch, I discovered that someone else fixed the bug in parallel.

This patch is a little better in that it consolidates the code.append()s into fewer calls.

Comment 3 Darin Fisher (:fishd, Google) 2009-03-13 17:14:46 PDT

Comment on attachment 28615 [details]
patch

LGTM

Comment 4 David Levin 2009-03-20 15:49:14 PDT

Assign to levin for landing.

Comment 5 David Levin 2009-03-20 16:57:12 PDT

Committed as r41880.