Bug 85659

Summary:

V8RecursionScope not declared in V8Proxy::newInstance which causes ASSERT() failure from NPAPI

Product:

WebKit

Reporter:

Rafael Weinstein <rafaelw>

Component:

Platform

Assignee:

Rafael Weinstein <rafaelw>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, adamk, fishd, haraken, japhet, ojan, rniwa, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	none

Description Rafael Weinstein 2012-05-04 13:42:24 PDT

newInstance should be treated as calling into page script just like instrumentedCallFunction

Comment 1 Rafael Weinstein 2012-05-04 13:44:52 PDT

Created attachment 140313 [details]
Patch

Comment 2 WebKit Review Bot 2012-05-04 14:21:08 PDT

Comment on attachment 140313 [details]
Patch

Clearing flags on attachment: 140313

Committed r116171: <http://trac.webkit.org/changeset/116171>

Comment 3 WebKit Review Bot 2012-05-04 14:21:13 PDT

All reviewed patches have been landed.  Closing bug.

Comment 4 Rafael Weinstein 2012-05-07 11:14:10 PDT

Reopening to attach new patch.

Comment 5 Rafael Weinstein 2012-05-07 11:14:13 PDT

Created attachment 140559 [details]
Patch

Comment 6 Ojan Vafai 2012-05-07 11:23:51 PDT

Pretty sure you meant this patch for a different bug. :)

Comment 7 Rafael Weinstein 2012-05-07 11:25:24 PDT

woops. yes