[builtins] Migrate SharedArrayBuffer.byteLength to C++.

[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive-by-fix: hydrogen code does not blindly return the
byteLength offset, instead it executes what is defined
in the byteLength getter.

BUG=

Committed: https://crrev.com/2f36ee71768a880a30a75caf4881ff27f71a26d2
Cr-Commit-Position: refs/heads/master@{#37595}

[Franzi, 2016-07-07 11:48:40]

Description was changed from

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.


BUG=
==========

to

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive by fix: optimized code returned the byteLength offset rather than do what
is defined in the byteLength getter. 

BUG=
==========

[Franzi, 2016-07-07 11:48:40]

franzih@chromium.org changed reviewers:
+ bmeurer@chromium.org

[Franzi, 2016-07-07 14:33:40]

Description was changed from

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive by fix: optimized code returned the byteLength offset rather than do what
is defined in the byteLength getter. 

BUG=
==========

to

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive-by-fix: optimized code does not blindly return the byteLength offset,
instead it executes what is defined in the byteLength getter. 

BUG=
==========

[Franzi, 2016-07-07 14:34:55]

Description was changed from

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive-by-fix: optimized code does not blindly return the byteLength offset,
instead it executes what is defined in the byteLength getter. 

BUG=
==========

to

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive-by-fix: optimized code does not blindly return the 
byteLength offset, instead it executes what is defined 
in the byteLength getter. 

BUG=
==========

[Franzi, 2016-07-07 14:36:26]

Description was changed from

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive-by-fix: optimized code does not blindly return the 
byteLength offset, instead it executes what is defined 
in the byteLength getter. 

BUG=
==========

to

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive-by-fix: hydrogen code does not blindly return the 
byteLength offset, instead it executes what is defined 
in the byteLength getter. 

BUG=
==========

[Franzi, 2016-07-07 14:38:32]

Hi Benedikt, 

Here's byteLength as C++ builtin for SharedArrayBuffers. We probably want
JS_SHARED_ARRAY_BUFFER_TYPE at some point instead of checking is_shared(). 

Please have a look. Thanks, 

Franzi

[Benedikt Meurer, 2016-07-07 19:00:50]

Very nice. LGTM, thanks.

[Franzi, 2016-07-07 19:11:24]

The CQ bit was checked by franzih@chromium.org

[commit-bot: I haz the power, 2016-07-07 19:11:35]

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

[commit-bot: I haz the power, 2016-07-07 19:42:21]

Description was changed from

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive-by-fix: hydrogen code does not blindly return the 
byteLength offset, instead it executes what is defined 
in the byteLength getter. 

BUG=
==========

to

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive-by-fix: hydrogen code does not blindly return the 
byteLength offset, instead it executes what is defined 
in the byteLength getter. 

BUG=
==========

[commit-bot: I haz the power, 2016-07-07 19:42:22]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2016-07-07 19:44:33]

Description was changed from

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive-by-fix: hydrogen code does not blindly return the 
byteLength offset, instead it executes what is defined 
in the byteLength getter. 

BUG=
==========

to

==========
[builtins] Migrate SharedArrayBuffer.byteLength to C++.

Drive-by-fix: hydrogen code does not blindly return the
byteLength offset, instead it executes what is defined
in the byteLength getter.

BUG=

Committed: https://crrev.com/2f36ee71768a880a30a75caf4881ff27f71a26d2
Cr-Commit-Position: refs/heads/master@{#37595}
==========

[commit-bot: I haz the power, 2016-07-07 19:44:34]

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