Issue 13412003: Demo for WebKit bug #113735

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 13412003: Demo for WebKit bug #113735 (Closed)

Created:
7 years, 8 months ago by Peter Rybin

Modified:
7 years, 5 months ago

Reviewers:
Yang

CC:
v8-dev

Base URL:
https://v8.googlecode.com/svn/branches/bleeding_edge

Visibility:
Public.

More Reviews

Description

Demo for WebKit bug #113735 See the bug info: https://bugs.webkit.org/show_bug.cgi?id=113735

Patch Set 1 #

Created: 7 years, 8 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+3 lines, -0 lines)			Patch
	M	src/array.js	View		1 chunk	+3 lines, -0 lines	0 comments	Download

Messages

Total messages: 1 (0 generated)

Expand Messages | Collapse Messages

Peter Rybin

7 years, 8 months ago (2013-04-02 00:36:22 UTC) #1

Hi Yang

Could you please advise with this issue.

Originally WebKit crashes on debug-mode assert at

receiver = isolate->factory()->ToObject(receiver,
calling_frames_native_context);

at runtime.cc:10443

because it spots a frame with a global object === undefined 

This patch checks that the undefined global object indeed can appear. If you
apply this patch and try to compile x64 version (not ia32!), build will fail.

We probably need to fix some part of this. Either get rid of undefined or allow
it in assert.

What do you think about this?

Peter

Expand Messages | Collapse Messages