Remove internal use of v8::AccessType, always pass v8::ACCESS_HAS instead.

Remove internal use of v8::AccessType, always pass v8::ACCESS_HAS instead.

BUG=

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

[Toon Verwaest, 2015-02-25 21:49:27]

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

[Toon Verwaest, 2015-02-25 21:49:27]

PTAL

[Toon Verwaest, 2015-02-25 21:52:15]

Actually, I just realized that for v8::ACCESS_KEYS we were passing in
undefined_value as the name. Do we need the name / index at all? Don't we just
disallow access to all or nothing?

[Toon Verwaest, 2015-02-25 21:52:46]

Additionally, do we need to distinguish between named/index access at all?

[Toon Verwaest, 2015-02-25 22:54:24]

Changed the CL to reflect this. From now we'll just always call the named
callback and ignore the indexed. The passed name is undefined.

Now we just need to add a new access-check-api, switch over, remove the old one.

[dcarney, 2015-02-26 08:08:00]

On 2015/02/25 22:54:24, Toon Verwaest wrote:
> Changed the CL to reflect this. From now we'll just always call the named
> callback and ignore the indexed. The passed name is undefined.
> 
> Now we just need to add a new access-check-api, switch over, remove the old
one.

lgtm, assuming blink isn't using the key anymore - do a quick check of google3
to make sure there's nothing to break there, but i can't imagine this is used.

[Toon Verwaest, 2015-02-26 10:05:50]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2015-02-26 10:13:04]

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

[commit-bot: I haz the power, 2015-02-26 10:34:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-26 10:35:01]

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