Support for typed arrays added to Heap::RightTrimFixedArray().

Support for typed arrays added to Heap::RightTrimFixedArray().

BUG=chromium:472513
LOG=Y

Committed: https://crrev.com/ffe886de4816312eece45fd7528d55807d3c51d9
Cr-Commit-Position: refs/heads/master@{#27586}

[Igor Sheludko, 2015-04-02 10:29:53]

ishell@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[Igor Sheludko, 2015-04-02 10:29:54]

PTAL

[Michael Starzinger, 2015-04-02 10:45:27]

LGTM.

https://codereview.chromium.org/1058793002/diff/1/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1058793002/diff/1/src/heap/heap.cc#newcode3617
src/heap/heap.cc:3617: Address new_end = object->address() + object->Size() -
bytes_to_trim;
nit: Can we keep the calculation outside where it was before? The common case is
that the object is not in LO-space, so the runtime impact doesn't matter. But
IMHO it keeps the code way more readable.

[Igor Sheludko, 2015-04-02 11:03:11]

https://codereview.chromium.org/1058793002/diff/1/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1058793002/diff/1/src/heap/heap.cc#newcode3617
src/heap/heap.cc:3617: Address new_end = object->address() + object->Size() -
bytes_to_trim;
On 2015/04/02 10:45:27, Michael Starzinger wrote:
> nit: Can we keep the calculation outside where it was before? The common case
is
> that the object is not in LO-space, so the runtime impact doesn't matter. But
> IMHO it keeps the code way more readable.

Done.

[Igor Sheludko, 2015-04-02 11:03:14]

The CQ bit was checked by ishell@chromium.org

[Igor Sheludko, 2015-04-02 11:03:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1058793002/#ps20001
(title: "Addressed comments")

[commit-bot: I haz the power, 2015-04-02 11:03:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1058793002/20001

[commit-bot: I haz the power, 2015-04-02 11:26:16]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-02 11:26:35]

Patchset 2 (id:??) landed as
https://crrev.com/ffe886de4816312eece45fd7528d55807d3c51d9
Cr-Commit-Position: refs/heads/master@{#27586}