Add V8.ArrayBufferLargeAllocations, deprecate V8.ArrayBufferBigAllocations.

Add V8.ArrayBufferLargeAllocations, deprecate V8.ArrayBufferBigAllocations.

CL https://codereview.chromium.org/2856663002 modifies V8 to only
track array buffer allocations only if the size is at least one
megabyte. This is being done because the UMA entry in
V8.ArrayBufferBigAllocations for sizes < 1Mb is so large (at least
99.93%) that the rest of the distribution was not visually visible.

To do this, the old table V8.ArrayBufferBigAllocations is marked
obsolete, and the new table V8.ArrayBufferLargeAllocations has
been added.

BUG=chrome:704922
Review-Url: https://codereview.chromium.org/2867483002
Cr-Commit-Position: refs/heads/master@{#469810}
Committed: https://chromium.googlesource.com/chromium/src/+/6031a48ccf1fa0ce3b62e3983e01a9b14d1260fe

[kschimpf, 2017-05-05 17:06:15]

Description was changed from

==========
Add V8.ArrayBufferLargeAllocations, deprecate V8.ArrayBufferBigAllocations.

CL https://codereview.chromium.org/2856663002 modifies V8 to only
track array buffer allocations only if the size is at least one
megabyte. This is being done because the UMA entry in
V8.ArrayBufferBigAllocations for sizes < 1Mb is so large (at least
99.93%) that the rest of the distribution was not visually visible.

To do this, the old table V8.ArrayBufferBigAllocations is marked
obsolete, and the new table V8.ArrayBufferLargeAllocations has
been added.

BUG=chrome:704922
==========

to

==========
Add V8.ArrayBufferLargeAllocations, deprecate V8.ArrayBufferBigAllocations.

CL https://codereview.chromium.org/2856663002 modifies V8 to only
track array buffer allocations only if the size is at least one
megabyte. This is being done because the UMA entry in
V8.ArrayBufferBigAllocations for sizes < 1Mb is so large (at least
99.93%) that the rest of the distribution was not visually visible.

To do this, the old table V8.ArrayBufferBigAllocations is marked
obsolete, and the new table V8.ArrayBufferLargeAllocations has
been added.

BUG=chrome:704922
==========

[kschimpf, 2017-05-05 17:06:16]

kschimpf@chromium.org changed reviewers:
+ bbudge@chromium.org, bradnelson@google.com, isherman@chromium.org

[bbudge, 2017-05-05 19:25:52]

lgtm w/nit

https://codereview.chromium.org/2867483002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2867483002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:77370: +    Number of bytes requested in
an array buffer allocations greater than one
s/an//

[Ilya Sherman, 2017-05-05 19:58:05]

LGTM

[kschimpf, 2017-05-05 20:01:13]

The CQ bit was checked by kschimpf@chromium.org

[commit-bot: I haz the power, 2017-05-05 20:01:47]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kschimpf, 2017-05-05 20:02:16]

The CQ bit was unchecked by kschimpf@chromium.org

[kschimpf, 2017-05-05 20:05:28]

The CQ bit was checked by kschimpf@chromium.org

[kschimpf, 2017-05-05 20:05:28]

The patchset sent to the CQ was uploaded after l-g-t-m from bbudge@chromium.org,
isherman@chromium.org
Link to the patchset: https://codereview.chromium.org/2867483002/#ps20001
(title: "Fix nit.")

[commit-bot: I haz the power, 2017-05-05 20:06:36]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kschimpf, 2017-05-05 20:47:09]

https://codereview.chromium.org/2867483002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2867483002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:77370: +    Number of bytes requested in
an array buffer allocations greater than one
On 2017/05/05 19:25:52, bbudge wrote:
> s/an//

Done.

[commit-bot: I haz the power, 2017-05-05 23:47:10]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494014728108220,
"parent_rev": "a9abb8f16cb14dd210cd5de4a7b6dbc2209059e6", "commit_rev":
"6031a48ccf1fa0ce3b62e3983e01a9b14d1260fe"}

[commit-bot: I haz the power, 2017-05-05 23:47:42]

Description was changed from

==========
Add V8.ArrayBufferLargeAllocations, deprecate V8.ArrayBufferBigAllocations.

CL https://codereview.chromium.org/2856663002 modifies V8 to only
track array buffer allocations only if the size is at least one
megabyte. This is being done because the UMA entry in
V8.ArrayBufferBigAllocations for sizes < 1Mb is so large (at least
99.93%) that the rest of the distribution was not visually visible.

To do this, the old table V8.ArrayBufferBigAllocations is marked
obsolete, and the new table V8.ArrayBufferLargeAllocations has
been added.

BUG=chrome:704922
==========

to

==========
Add V8.ArrayBufferLargeAllocations, deprecate V8.ArrayBufferBigAllocations.

CL https://codereview.chromium.org/2856663002 modifies V8 to only
track array buffer allocations only if the size is at least one
megabyte. This is being done because the UMA entry in
V8.ArrayBufferBigAllocations for sizes < 1Mb is so large (at least
99.93%) that the rest of the distribution was not visually visible.

To do this, the old table V8.ArrayBufferBigAllocations is marked
obsolete, and the new table V8.ArrayBufferLargeAllocations has
been added.

BUG=chrome:704922

Review-Url: https://codereview.chromium.org/2867483002
Cr-Commit-Position: refs/heads/master@{#469810}
Committed:
https://chromium.googlesource.com/chromium/src/+/6031a48ccf1fa0ce3b62e3983e01...
==========

[commit-bot: I haz the power, 2017-05-05 23:47:44]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/6031a48ccf1fa0ce3b62e3983e01...