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

Issue 518056: Added ES5 15.2.3.2 Object.getPrototypeOf. (Closed)

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

Description

Added ES5 15.2.3.2 Object.getPrototypeOf. Committed: http://code.google.com/p/v8/source/detail?r=3551

Patch Set 1 #

Total comments: 2

Patch Set 2 : '' #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+85 lines, -4 lines) Patch
M src/messages.js View 1 chunk +2 lines, -1 line 0 comments Download
M src/v8natives.js View 1 2 chunks +11 lines, -1 line 0 comments Download
M test/es5conform/es5conform.status View 2 chunks +3 lines, -1 line 0 comments Download
A test/mjsunit/get-prototype-of.js View 1 1 chunk +68 lines, -0 lines 0 comments Download
M tools/presubmit.py View 1 chunk +1 line, -1 line 2 comments Download

Messages

Total messages: 4 (0 generated)
Rico
10 years, 11 months ago (2010-01-07 08:28:13 UTC) #1
Christian Plesner Hansen
LGTM http://codereview.chromium.org/518056/diff/1/3 File src/v8natives.js (right): http://codereview.chromium.org/518056/diff/1/3#newcode453 src/v8natives.js:453: if (!IS_OBJECT(obj) && !IS_FUNCTION(obj)) { How about !(IS_OBJECT(obj) ...
10 years, 11 months ago (2010-01-07 08:32:11 UTC) #2
Lasse Reichstein
Drive-by comment. http://codereview.chromium.org/518056/diff/4001/4006 File tools/presubmit.py (right): http://codereview.chromium.org/518056/diff/4001/4006#newcode224 tools/presubmit.py:224: r'Copyright [\d-]*20[0-1][0-9] the V8 project authors. All ...
10 years, 11 months ago (2010-01-07 09:39:14 UTC) #3
Christian Plesner Hansen
10 years, 11 months ago (2010-01-07 09:42:48 UTC) #4
http://codereview.chromium.org/518056/diff/4001/4006
File tools/presubmit.py (right):

http://codereview.chromium.org/518056/diff/4001/4006#newcode224
tools/presubmit.py:224: r'Copyright [\d-]*20[0-1][0-9] the V8 project authors.
All rights reserved.')
I think [0-1][0-9] is a good compromise between potentially catching mistakes
without making the regexp overly complex.  But I don't have particularly strong
feelings about it.

Powered by Google App Engine
This is Rietveld 408576698