Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(190)

Issue 545109: Added ES5 conform tests for getOwnPropertyNames.... (Closed)

Created:
10 years, 11 months ago by Rico
Modified:
9 years, 6 months ago
CC:
v8-dev
Visibility:
Public.

Description

Added ES5 conform tests for getOwnPropertyNames. Most of the test cases fail as the different objects according to the tests can not have additional properties attached to them. I will file a bug report on the es5 conform site as they should allow this. Some of the test fails because we still miss some of the es5 features used. Committed: http://code.google.com/p/v8/source/detail?r=3642

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+111 lines, -1 line) Patch
M test/es5conform/es5conform.status View 2 chunks +111 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Rico
10 years, 11 months ago (2010-01-18 12:31:09 UTC) #1
Søren Thygesen Gjesse
LGTM I think you should raise a bug to track the missing pieces of getOwnPropertyNames.
10 years, 11 months ago (2010-01-18 14:27:42 UTC) #2
Rico
10 years, 11 months ago (2010-01-19 08:22:44 UTC) #3
On 2010/01/18 14:27:42, Søren Gjesse wrote:
> LGTM
> 
> I think you should raise a bug to track the missing pieces of
> getOwnPropertyNames.
Done

Powered by Google App Engine
This is Rietveld 408576698