Bug 35710

Summary:

[chromium] DateExtension fires incorrectly

Product:

WebKit

Reporter:

John Abd-El-Malek <jam>

Component:

Platform

Assignee:

John Abd-El-Malek <jam>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, dglazkov, fishd, ojan

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Proposed patch	none

Description John Abd-El-Malek 2010-03-03 17:04:44 PST

Full background here: http://code.google.com/p/chromium/issues/detail?id=33441#makechanges

The problem was this line:
argv[0] = v8::String::New(allow ? "false" : "true");

since the JS function did "if (enable)" and treated the string as bool, it was always true.

Comment 1 John Abd-El-Malek 2010-03-03 17:11:47 PST

Created attachment 49965 [details]
Proposed patch

Comment 2 Adam Barth 2010-03-03 17:23:39 PST

Comment on attachment 49965 [details]
Proposed patch

Type FTW

Comment 3 John Abd-El-Malek 2010-03-03 17:31:52 PST

Thanks for the quick review!

Comment 4 WebKit Commit Bot 2010-03-04 01:24:52 PST

Comment on attachment 49965 [details]
Proposed patch

Clearing flags on attachment: 49965

Committed r55513: <http://trac.webkit.org/changeset/55513>

Comment 5 WebKit Commit Bot 2010-03-04 01:24:56 PST

All reviewed patches have been landed.  Closing bug.