Bug 93220

Summary:

[V8] Replace all V8 undefined values in the rest of V8 bindings with v8Undefined()

Product:

WebKit

Reporter:

Kentaro Hara <haraken>

Component:

WebCore JavaScript

Assignee:

Kentaro Hara <haraken>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, japhet, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

93095

Attachments:

Description	Flags
Patch	none
Patch	none

Description Kentaro Hara 2012-08-05 19:15:25 PDT

We should use v8Undefined() everywhere in V8 bindings.

In this bug, we replace all V8 undefined values in the rest of custom bindings with v8Undefined().

Comment 1 Kentaro Hara 2012-08-05 19:18:52 PDT

Created attachment 156585 [details]
Patch

Comment 2 Kentaro Hara 2012-08-05 19:51:15 PDT

Created attachment 156586 [details]
Patch

Comment 3 Eric Seidel (no email) 2012-08-07 15:21:33 PDT

Comment on attachment 156586 [details]
Patch

LGTM.

Comment 4 WebKit Review Bot 2012-08-07 16:32:06 PDT

Comment on attachment 156586 [details]
Patch

Clearing flags on attachment: 156586

Committed r124938: <http://trac.webkit.org/changeset/124938>

Comment 5 WebKit Review Bot 2012-08-07 16:32:10 PDT

All reviewed patches have been landed.  Closing bug.