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

Issue 346233002: Add a use counter API (Closed)

Created:
6 years, 6 months ago by jochen (gone - plz use gerrit)
Modified:
6 years, 6 months ago
Reviewers:
Sven Panne, marja
CC:
v8-dev, Paweł Hajdan Jr.
Visibility:
Public.

Description

Add a use counter API This lets embedders track certain features of v8 and the number of times they are used BUG=none R=svenpanne@chromium.org, marja@chromium.org LOG=y Committed: https://code.google.com/p/v8/source/detail?r=21925

Patch Set 1 #

Total comments: 3

Patch Set 2 : updates #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -0 lines) Patch
M include/v8.h View 2 chunks +18 lines, -0 lines 0 comments Download
M src/api.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M src/isolate.h View 2 chunks +5 lines, -0 lines 0 comments Download
M src/isolate.cc View 1 1 chunk +13 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
jochen (gone - plz use gerrit)
6 years, 6 months ago (2014-06-23 09:05:18 UTC) #1
Sven Panne
https://codereview.chromium.org/346233002/diff/1/include/v8.h File include/v8.h (right): https://codereview.chromium.org/346233002/diff/1/include/v8.h#newcode4132 include/v8.h:4132: * Features reported via the SetUseCounterCallback callback. Do not ...
6 years, 6 months ago (2014-06-23 09:29:39 UTC) #2
jochen (gone - plz use gerrit)
updated, ptal
6 years, 6 months ago (2014-06-23 09:37:29 UTC) #3
Sven Panne
LGTM, Chrome uses a similar enum.
6 years, 6 months ago (2014-06-23 09:37:36 UTC) #4
jochen (gone - plz use gerrit)
6 years, 6 months ago (2014-06-23 09:47:07 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r21925 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698