Move the contents of api-natives.js to c++

Move the contents of api-natives.js to c++

R=verwaest@chromium.org

BUG=

Committed: https://crrev.com/8064582626f4f2e012352750fda839ae3f975127
Cr-Commit-Position: refs/heads/master@{#26426}

[dcarney, 2015-02-04 10:48:50]

dcarney@chromium.org changed reviewers:
+ verwaest@google.com

[dcarney, 2015-02-04 10:49:03]

ptal

[dcarney, 2015-02-04 10:49:19]

dcarney@chromium.org changed reviewers:
+ verwaest@chromium.org
- verwaest@google.com

[dcarney, 2015-02-04 10:49:25]

ptal

[Toon Verwaest, 2015-02-04 12:04:05]

Very nice cleanup.
lgtm with minor comments

https://codereview.chromium.org/895053002/diff/300001/src/api-natives.cc
File src/api-natives.cc (right):

https://codereview.chromium.org/895053002/diff/300001/src/api-natives.cc#newc...
src/api-natives.cc:133: // standard.
Would indeed be nice if you'd just move the Disable/Enable access checks out of
this function in a wrapper.

https://codereview.chromium.org/895053002/diff/300001/src/api-natives.cc#newc...
src/api-natives.cc:148: AddPropertyForTemplate(isolate, obj, name, value,
attributes);
Can you change this to return a MaybeHandle<> rather than checking IsException?
That's the C++ side of the exception API.

https://codereview.chromium.org/895053002/diff/300001/src/api-natives.cc#newc...
src/api-natives.cc:162: setter, attributes);
Same here

https://codereview.chromium.org/895053002/diff/300001/src/api-natives.cc#newc...
src/api-natives.cc:205: Handle<FixedArray>
cache(isolate->native_context()->function_cache());
auto cache = isolate->function_cache();

https://codereview.chromium.org/895053002/diff/300001/src/api-natives.cc#newc...
src/api-natives.cc:206: if
(isolate->native_context()->function_cache()->length() <= serial_number) {
cache->length()

https://codereview.chromium.org/895053002/diff/300001/src/api-natives.cc#newc...
src/api-natives.cc:226: Handle<FixedArray>
cache(isolate->native_context()->function_cache());
auto cache = isolate->function_cache()

[dcarney, 2015-02-04 12:36:27]

New patchsets have been uploaded after l-g-t-m from verwaest@chromium.org

[dcarney, 2015-02-04 12:39:36]

The CQ bit was checked by dcarney@chromium.org

[commit-bot: I haz the power, 2015-02-04 12:40:05]

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

[commit-bot: I haz the power, 2015-02-04 13:01:41]

Committed patchset #18 (id:340001)

[commit-bot: I haz the power, 2015-02-04 13:01:56]

Patchset 18 (id:??) landed as
https://crrev.com/8064582626f4f2e012352750fda839ae3f975127
Cr-Commit-Position: refs/heads/master@{#26426}