On 2015/03/31 18:19:32, adamk wrote:
Where's the Blink side of this change?
I'll get to it once this rolls.
https://codereview.chromium.org/1041863002/diff/20001/src/parser.cc
File src/parser.cc (right):
https://codereview.chromium.org/1041863002/diff/20001/src/parser.cc#newcode2275
src/parser.cc:2275: ++use_counts_[v8::Isolate::kLegacyConst];
Counting each const in a script separately seems a bit odd, especially
since
Blink will collapse all these into one. But I guess that's not
particularly
problematic. Just wondering if it's intentional that the V8 API for
UseCounters
doesn't match how they're used in Blink.
Yes, it is intentional. Embedder is free to collapse use counters in any
way it
wants, we still report every usage.
https://codereview.chromium.org/1041863002/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
