[runtime] Rename IsUndetectableObject to IsUndetectable.

[runtime] Rename IsUndetectableObject to IsUndetectable.

This is more consistent with the current naming scheme (i.e. IsCallable
for callable bit on map, IsConstructor for constructor bit on map, and
now IsUndetectable for undetectable bit on map).

Also simplify the fallthrough case for Object::Equals, because we don't
need to check for Null or Undefined or Undetectable, as both Null and
Undefined already have the undetectable bit set on their maps.

R=machenbach@chromium.org

Committed: https://crrev.com/0b3e436aa5e66e425d63fa725acbc1bbc0d7cc8c
Cr-Commit-Position: refs/heads/master@{#34458}

[Benedikt Meurer, 2016-03-03 07:58:05]

Hey Michael,

Easy refactoring and a simplification for Object::Equals.
Please take a look.

Thanks,
Benedikt

[Benedikt Meurer, 2016-03-03 08:00:47]

Description was changed from

==========
[runtime] Rename IsUndetectableObject to IsUndetectable.

This is more consistent with the current naming scheme (i.e. IsCallable
for callable bit on map, IsConstructor for constructor bit on map, and
now IsUndetect for undetectable bit on map).

Also simplify the fallthrough case for Object::Equals, because we don't
need to check for Null or Undefined or Undetectable, as both Null and
Undefined already have the undetectable bit set on their maps.

R=machenbach@chromium.org
==========

to

==========
[runtime] Rename IsUndetectableObject to IsUndetectable.

This is more consistent with the current naming scheme (i.e. IsCallable
for callable bit on map, IsConstructor for constructor bit on map, and
now IsUndetectable for undetectable bit on map).

Also simplify the fallthrough case for Object::Equals, because we don't
need to check for Null or Undefined or Undetectable, as both Null and
Undefined already have the undetectable bit set on their maps.

R=machenbach@chromium.org
==========

[Michael Achenbach, 2016-03-03 08:07:06]

lgtm

https://codereview.chromium.org/1756413003/diff/1/src/objects.cc
File src/objects.cc (left):

https://codereview.chromium.org/1756413003/diff/1/src/objects.cc#oldcode374
src/objects.cc:374: (x->IsNull() || x->IsUndefined() ||
x->IsUndetectableObject()) &&
This change is rubber-stamped.

[Michael Achenbach, 2016-03-03 08:09:23]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-03-03 08:09:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1756413003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1756413003/1

[Benedikt Meurer, 2016-03-03 08:09:45]

https://codereview.chromium.org/1756413003/diff/1/src/objects.cc
File src/objects.cc (left):

https://codereview.chromium.org/1756413003/diff/1/src/objects.cc#oldcode374
src/objects.cc:374: (x->IsNull() || x->IsUndefined() ||
x->IsUndetectableObject()) &&
This is the same change that Toon did earlier for CompareIC::GenerateGeneric.

[commit-bot: I haz the power, 2016-03-03 08:24:44]

Description was changed from

==========
[runtime] Rename IsUndetectableObject to IsUndetectable.

This is more consistent with the current naming scheme (i.e. IsCallable
for callable bit on map, IsConstructor for constructor bit on map, and
now IsUndetectable for undetectable bit on map).

Also simplify the fallthrough case for Object::Equals, because we don't
need to check for Null or Undefined or Undetectable, as both Null and
Undefined already have the undetectable bit set on their maps.

R=machenbach@chromium.org
==========

to

==========
[runtime] Rename IsUndetectableObject to IsUndetectable.

This is more consistent with the current naming scheme (i.e. IsCallable
for callable bit on map, IsConstructor for constructor bit on map, and
now IsUndetectable for undetectable bit on map).

Also simplify the fallthrough case for Object::Equals, because we don't
need to check for Null or Undefined or Undetectable, as both Null and
Undefined already have the undetectable bit set on their maps.

R=machenbach@chromium.org
==========

[commit-bot: I haz the power, 2016-03-03 08:24:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-03 08:26:12]

Description was changed from

==========
[runtime] Rename IsUndetectableObject to IsUndetectable.

This is more consistent with the current naming scheme (i.e. IsCallable
for callable bit on map, IsConstructor for constructor bit on map, and
now IsUndetectable for undetectable bit on map).

Also simplify the fallthrough case for Object::Equals, because we don't
need to check for Null or Undefined or Undetectable, as both Null and
Undefined already have the undetectable bit set on their maps.

R=machenbach@chromium.org
==========

to

==========
[runtime] Rename IsUndetectableObject to IsUndetectable.

This is more consistent with the current naming scheme (i.e. IsCallable
for callable bit on map, IsConstructor for constructor bit on map, and
now IsUndetectable for undetectable bit on map).

Also simplify the fallthrough case for Object::Equals, because we don't
need to check for Null or Undefined or Undetectable, as both Null and
Undefined already have the undetectable bit set on their maps.

R=machenbach@chromium.org

Committed: https://crrev.com/0b3e436aa5e66e425d63fa725acbc1bbc0d7cc8c
Cr-Commit-Position: refs/heads/master@{#34458}
==========

[commit-bot: I haz the power, 2016-03-03 08:26:13]

Patchset 1 (id:??) landed as
https://crrev.com/0b3e436aa5e66e425d63fa725acbc1bbc0d7cc8c
Cr-Commit-Position: refs/heads/master@{#34458}