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

Issue 243044: Update V8 to version 1.3.13.4.... (Closed)

Created:
11 years, 2 months ago by Søren Thygesen Gjesse
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Update V8 to version 1.3.13.4. This version contains a fix for a V8 bug (actually in version 1.3.13.3) and adds some missing handle scopes in the API. BUG=22913 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=27604

Patch Set 1 #

Total comments: 2

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Søren Thygesen Gjesse
11 years, 2 months ago (2009-09-30 10:40:45 UTC) #1
Mads Ager (chromium)
http://codereview.chromium.org/243044/diff/1/2 File DEPS (right): http://codereview.chromium.org/243044/diff/1/2#newcode40 Line 40: "http://v8.googlecode.com/svn/trunk@2988", Shouldn't it be 2990 or something?
11 years, 2 months ago (2009-09-30 10:42:37 UTC) #2
Søren Thygesen Gjesse
http://codereview.chromium.org/243044/diff/1/2 File DEPS (right): http://codereview.chromium.org/243044/diff/1/2#newcode40 Line 40: "http://v8.googlecode.com/svn/trunk@2988", On 2009/09/30 10:42:37, Mads Ager wrote: > ...
11 years, 2 months ago (2009-09-30 11:00:04 UTC) #3
Mads Ager (chromium)
11 years, 2 months ago (2009-09-30 11:00:38 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698