[rutime] Simplify undetectable handling in Object::Equals.

[rutime] Simplify undetectable handling in Object::Equals.

According to https://www.w3.org/TR/html5/obsolete.html#dom-document-all
the undetectable bit (for document.all) only affects comparisons with
undefined and null. In particular comparisons with other values are not
affected, so we can actually simplify the handling of undetectable a
lot by only checking it when null or undefined might be involved (this
is actually in line with what the CompareIC does).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
R=verwaest@chromium.org

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

[Benedikt Meurer, 2016-03-04 07:42:06]

Hey Toon,

Simplify the Object::Equals primitive a bit wrt. undetectable.
Please take a look.

Thanks,
Benedikt

[Benedikt Meurer, 2016-03-04 07:42:33]

Description was changed from

==========
[rutime] Simplify undetectable handling in Object::Equals.

According to https://www.w3.org/TR/html5/obsolete.html#dom-document-all
the undetectable bit (for document.all) only affects comparisons with
undefined and null. In particular comparisons with other values are not
affected, so we can actually simplify the handling of undetectable a
lot by only checking it when null or undefined might be involved (this
is actually in line with what the CompareIC does).

R=verwaest@chromium.org
==========

to

==========
[rutime] Simplify undetectable handling in Object::Equals.

According to https://www.w3.org/TR/html5/obsolete.html#dom-document-all
the undetectable bit (for document.all) only affects comparisons with
undefined and null. In particular comparisons with other values are not
affected, so we can actually simplify the handling of undetectable a
lot by only checking it when null or undefined might be involved (this
is actually in line with what the CompareIC does).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
R=verwaest@chromium.org
==========

[Benedikt Meurer, 2016-03-04 07:42:41]

The CQ bit was checked by bmeurer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-04 07:42:44]

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

[Toon Verwaest, 2016-03-04 07:44:49]

lgtm

[Benedikt Meurer, 2016-03-04 07:46:55]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-03-04 07:47:02]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-03-04 07:47:04]

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

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

Description was changed from

==========
[rutime] Simplify undetectable handling in Object::Equals.

According to https://www.w3.org/TR/html5/obsolete.html#dom-document-all
the undetectable bit (for document.all) only affects comparisons with
undefined and null. In particular comparisons with other values are not
affected, so we can actually simplify the handling of undetectable a
lot by only checking it when null or undefined might be involved (this
is actually in line with what the CompareIC does).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
R=verwaest@chromium.org
==========

to

==========
[rutime] Simplify undetectable handling in Object::Equals.

According to https://www.w3.org/TR/html5/obsolete.html#dom-document-all
the undetectable bit (for document.all) only affects comparisons with
undefined and null. In particular comparisons with other values are not
affected, so we can actually simplify the handling of undetectable a
lot by only checking it when null or undefined might be involved (this
is actually in line with what the CompareIC does).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
R=verwaest@chromium.org
==========

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

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-04 09:09:42]

Description was changed from

==========
[rutime] Simplify undetectable handling in Object::Equals.

According to https://www.w3.org/TR/html5/obsolete.html#dom-document-all
the undetectable bit (for document.all) only affects comparisons with
undefined and null. In particular comparisons with other values are not
affected, so we can actually simplify the handling of undetectable a
lot by only checking it when null or undefined might be involved (this
is actually in line with what the CompareIC does).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
R=verwaest@chromium.org
==========

to

==========
[rutime] Simplify undetectable handling in Object::Equals.

According to https://www.w3.org/TR/html5/obsolete.html#dom-document-all
the undetectable bit (for document.all) only affects comparisons with
undefined and null. In particular comparisons with other values are not
affected, so we can actually simplify the handling of undetectable a
lot by only checking it when null or undefined might be involved (this
is actually in line with what the CompareIC does).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
R=verwaest@chromium.org

Committed: https://crrev.com/e0750529760de8a1bfe344dd5c0d92a9ac453849
Cr-Commit-Position: refs/heads/master@{#34483}
==========

[commit-bot: I haz the power, 2016-03-04 09:09:43]

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