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

Issue 122073003: Track v8 API changes. (Closed)

Created:
6 years, 11 months ago by Sven Panne
Modified:
6 years, 11 months ago
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -8 lines) Patch
M chrome/test/base/v8_unit_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/public/test/render_view_test.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M content/renderer/v8_value_converter_impl_unittest.cc View 4 chunks +5 lines, -5 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Sven Panne
PTAL and tick the 'Commit:' box... :-)
6 years, 11 months ago (2014-01-03 20:00:35 UTC) #1
jochen (gone - plz use gerrit)
lgtm
6 years, 11 months ago (2014-01-03 20:03:57 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/svenpanne@chromium.org/122073003/1
6 years, 11 months ago (2014-01-03 20:04:11 UTC) #3
jochen (gone - plz use gerrit)
Committed patchset #1 manually as r242916 (presubmit successful).
6 years, 11 months ago (2014-01-03 20:12:08 UTC) #4
esprehn
6 years, 11 months ago (2014-01-03 22:01:49 UTC) #5
Message was sent while issue was closed.
On 2014/01/03 20:12:08, jochen wrote:
> Committed patchset #1 manually as r242916 (presubmit successful).

I think you broke the build:
http://build.chromium.org/p/chromium/builders/Mac/builds/24229

Powered by Google App Engine
This is Rietveld 408576698