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

Issue 10166007: Add names to GlobalObject inline properties in heap snapshot. (Closed)

Created:
8 years, 8 months ago by alexeif
Modified:
8 years, 8 months ago
CC:
v8-dev
Visibility:
Public.

Description

Add names to GlobalObject inline properties in heap snapshot. Committed: https://code.google.com/p/v8/source/detail?r=11410

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -0 lines) Patch
M src/profile-generator.cc View 1 chunk +11 lines, -0 lines 3 comments Download

Messages

Total messages: 5 (0 generated)
alexeif
8 years, 8 months ago (2012-04-20 17:27:01 UTC) #1
mnaganov (inactive)
lgtm
8 years, 8 months ago (2012-04-20 19:19:49 UTC) #2
yurys
https://chromiumcodereview.appspot.com/10166007/diff/1/src/profile-generator.cc File src/profile-generator.cc (right): https://chromiumcodereview.appspot.com/10166007/diff/1/src/profile-generator.cc#newcode2048 src/profile-generator.cc:2048: SetInternalReference(global_obj, entry, Is it possible to merge such objects ...
8 years, 8 months ago (2012-04-23 05:36:18 UTC) #3
alexeif
https://chromiumcodereview.appspot.com/10166007/diff/1/src/profile-generator.cc File src/profile-generator.cc (right): https://chromiumcodereview.appspot.com/10166007/diff/1/src/profile-generator.cc#newcode2048 src/profile-generator.cc:2048: SetInternalReference(global_obj, entry, On 2012/04/23 05:36:18, Yury Semikhatsky wrote: > ...
8 years, 8 months ago (2012-04-23 15:23:43 UTC) #4
yurys
8 years, 8 months ago (2012-04-24 08:38:50 UTC) #5
https://chromiumcodereview.appspot.com/10166007/diff/1/src/profile-generator.cc
File src/profile-generator.cc (right):

https://chromiumcodereview.appspot.com/10166007/diff/1/src/profile-generator....
src/profile-generator.cc:2048: SetInternalReference(global_obj, entry,
On 2012/04/23 15:23:43, alexeif wrote:
> On 2012/04/23 05:36:18, Yury Semikhatsky wrote:
> > Is it possible to merge such objects into the global one (in the snapshot)
to
> > avoid user confusion?
> Could you please elaborate?

I meant that it may be hard to explain to the user what things like
global_receiver on the global object are. I understand that it is accurate
representation of what's going on under the hood but since we are trying to
filter out some internal objects that would make sense to fix things like this
as well.

Powered by Google App Engine
This is Rietveld 408576698