Introduce NonJSProxyHolder returning Handle<JSObject> and return Handle<JSReceiver> for GetHolder

Introduce NonJSProxyHolder returning Handle<JSObject> and return Handle<JSReceiver> for GetHolder

BUG=
R=rossberg@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=22594

[Toon Verwaest, 2014-07-24 11:08:05]

PTAL

[rossberg, 2014-07-24 11:18:33]

https://codereview.chromium.org/415953005/diff/1/src/lookup.h
File src/lookup.h (right):

https://codereview.chromium.org/415953005/diff/1/src/lookup.h#newcode100
src/lookup.h:100: Handle<JSObject> GetNonJSProxyHolder() const {
You should call it GetJSObjectHolder.

But I'm not sure I like such random downcast accessors all that much. How about
changing GetHolder to

template<class T>
Handle<T> GetHolder() const {
  ASSERT(IsFound());
  return Handle<T>::cast(maybe_holder_.ToHandleChecked());
}

[Toon Verwaest, 2014-07-24 11:25:47]

PTAL again

[rossberg, 2014-07-24 11:32:35]

lgtm

[Toon Verwaest, 2014-07-24 11:33:54]

Committed patchset #3 manually as r22594 (presubmit successful).

[arv (Not doing code reviews), 2014-07-24 17:01:54]

LGTM

Thanks for fixing this.