Fix v8::Object::DeleteHiddenValue to not bail when there are no hidden proper...

Fix v8::Object::DeleteHiddenValue to not bail when there are no hidden properties.

Also make it return true even if there was nothing to delete, to match the
behavior of v8::Object::Delete. 

This is to fix http://code.google.com/p/v8/issues/detail?id=300

[iposva, 2009-04-08 04:52:04]

One comment, otherwise LGTM!

Thanks,
-Ivan

http://codereview.chromium.org/58016/diff/1/2
File src/api.cc (right):

http://codereview.chromium.org/58016/diff/1/2#newcode2085
Line 2085: if (js_object->IsUndefined()) {
This test seems redundant. hidden_props has to be !undefined to reach here, and
the cast from i::Object to i::JSObject is definitely not going to make an
undefined value.

[asargent_no_longer_on_chrome, 2009-04-08 20:28:26]

http://codereview.chromium.org/58016/diff/1/2
File src/api.cc (right):

http://codereview.chromium.org/58016/diff/1/2#newcode2085
Line 2085: if (js_object->IsUndefined()) {
On 2009/04/08 04:52:04, iposva wrote:
> This test seems redundant. hidden_props has to be !undefined to reach here,
and
> the cast from i::Object to i::JSObject is definitely not going to make an
> undefined value.

Done.

I also renamed 'js_object' to 'js_obj' to better match the surrounding code's
naming conventions.