Use the LookupIterator in SetAccessor

Use the LookupIterator in SetAccessor

BUG=v8:4137
LOG=n

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

[Toon Verwaest, 2015-06-11 12:02:58]

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

[Toon Verwaest, 2015-06-11 12:02:59]

ptal

I additionally updated the related test to except exceptions to be thrown.
Testing the alternative default behavior is useless as it is used by no-one. We
should always throw in the future, throwing ourselves if the callback didn't.

[Jakob Kummerow, 2015-06-11 13:39:25]

lgtm

[Toon Verwaest, 2015-06-11 16:30:39]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2015-06-11 16:30:46]

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

[commit-bot: I haz the power, 2015-06-11 17:03:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-11 17:04:07]

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

[Igor Sheludko, 2015-06-11 17:19:38]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1175323004/ by ishell@chromium.org.

The reason for reverting is: Blocks reverting of
https://codereview.chromium.org/1175973002.