Revert "Speed up the LookupIterator"

Revert "Speed up the LookupIterator"

This reverts commit 2608ecc7157a9d6d482709fc197b2fc0317d9d09.

Revert "Specialize helper methods in the LookupIterator by is_element."

This reverts commit 6eb483f8782e32d32f75d091fea4b50846210372.

Revert "Avoid SetPropertyInternal if the LookupIterator is NotFound"

This reverts commit ca5bd8d4a9d91336d770668dc569661e4fba5aa3.

Revert "Inline fast-bailout-checks for LookupIterator::UpdateProtector"

This reverts commit d98570a1eb7f017ca6ecc9d79236456f2691aee3.

This breaks layout tests with timeouts:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/5060

It also seems to break jsfunfuzz:
https://build.chromium.org/p/client.v8/builders/V8%20Fuzzer/builds/7930

The other three CLs are reverted to be able to revert the first.

BUG=v8:4798
LOG=n
TBR=verwaest@chromium.org

Committed: https://crrev.com/70ac41a84a1dbb2025bb447a81582146ceb2a5ff
Cr-Commit-Position: refs/heads/master@{#34457}

[Michael Achenbach, 2016-03-03 07:41:15]

Description was changed from

==========
Revert "Speed up the LookupIterator"

This reverts commit 2608ecc7157a9d6d482709fc197b2fc0317d9d09.

Revert "Specialize helper methods in the LookupIterator by is_element."

This reverts commit 6eb483f8782e32d32f75d091fea4b50846210372.

Revert "Avoid SetPropertyInternal if the LookupIterator is NotFound"

This reverts commit ca5bd8d4a9d91336d770668dc569661e4fba5aa3.

Revert "Inline fast-bailout-checks for LookupIterator::UpdateProtector"

This reverts commit d98570a1eb7f017ca6ecc9d79236456f2691aee3.

BUG=
==========

to

==========
Revert "Speed up the LookupIterator"

This reverts commit 2608ecc7157a9d6d482709fc197b2fc0317d9d09.

Revert "Specialize helper methods in the LookupIterator by is_element."

This reverts commit 6eb483f8782e32d32f75d091fea4b50846210372.

Revert "Avoid SetPropertyInternal if the LookupIterator is NotFound"

This reverts commit ca5bd8d4a9d91336d770668dc569661e4fba5aa3.

Revert "Inline fast-bailout-checks for LookupIterator::UpdateProtector"

This reverts commit d98570a1eb7f017ca6ecc9d79236456f2691aee3.


This breaks layout tests with timeouts:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

It also seems to break jsfunfuzz:
https://build.chromium.org/p/client.v8/builders/V8%20Fuzzer/builds/7930

The other three CLs are reverted to be able to revert the first.

BUG=v8:4798
TBR=verwaest@chromium.org
==========

[Michael Achenbach, 2016-03-03 07:41:15]

machenbach@chromium.org changed reviewers:
+ jkummerow@chromium.org, jochen@chromium.org, verwaest@chromium.org

[Michael Achenbach, 2016-03-03 07:41:36]

TBR

[Michael Achenbach, 2016-03-03 07:41:47]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-03-03 07:41:52]

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

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

The CQ bit was unchecked by commit-bot@chromium.org

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

Try jobs failed on following builders:
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/11821)

[Michael Achenbach, 2016-03-03 08:08:35]

Description was changed from

==========
Revert "Speed up the LookupIterator"

This reverts commit 2608ecc7157a9d6d482709fc197b2fc0317d9d09.

Revert "Specialize helper methods in the LookupIterator by is_element."

This reverts commit 6eb483f8782e32d32f75d091fea4b50846210372.

Revert "Avoid SetPropertyInternal if the LookupIterator is NotFound"

This reverts commit ca5bd8d4a9d91336d770668dc569661e4fba5aa3.

Revert "Inline fast-bailout-checks for LookupIterator::UpdateProtector"

This reverts commit d98570a1eb7f017ca6ecc9d79236456f2691aee3.


This breaks layout tests with timeouts:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

It also seems to break jsfunfuzz:
https://build.chromium.org/p/client.v8/builders/V8%20Fuzzer/builds/7930

The other three CLs are reverted to be able to revert the first.

BUG=v8:4798
TBR=verwaest@chromium.org
==========

to

==========
Revert "Speed up the LookupIterator"

This reverts commit 2608ecc7157a9d6d482709fc197b2fc0317d9d09.

Revert "Specialize helper methods in the LookupIterator by is_element."

This reverts commit 6eb483f8782e32d32f75d091fea4b50846210372.

Revert "Avoid SetPropertyInternal if the LookupIterator is NotFound"

This reverts commit ca5bd8d4a9d91336d770668dc569661e4fba5aa3.

Revert "Inline fast-bailout-checks for LookupIterator::UpdateProtector"

This reverts commit d98570a1eb7f017ca6ecc9d79236456f2691aee3.


This breaks layout tests with timeouts:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

It also seems to break jsfunfuzz:
https://build.chromium.org/p/client.v8/builders/V8%20Fuzzer/builds/7930

The other three CLs are reverted to be able to revert the first.

BUG=v8:4798
LOG=n
TBR=verwaest@chromium.org
==========

[Michael Achenbach, 2016-03-03 08:08:40]

The CQ bit was checked by machenbach@chromium.org

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

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

[commit-bot: I haz the power, 2016-03-03 08:10:49]

Description was changed from

==========
Revert "Speed up the LookupIterator"

This reverts commit 2608ecc7157a9d6d482709fc197b2fc0317d9d09.

Revert "Specialize helper methods in the LookupIterator by is_element."

This reverts commit 6eb483f8782e32d32f75d091fea4b50846210372.

Revert "Avoid SetPropertyInternal if the LookupIterator is NotFound"

This reverts commit ca5bd8d4a9d91336d770668dc569661e4fba5aa3.

Revert "Inline fast-bailout-checks for LookupIterator::UpdateProtector"

This reverts commit d98570a1eb7f017ca6ecc9d79236456f2691aee3.


This breaks layout tests with timeouts:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

It also seems to break jsfunfuzz:
https://build.chromium.org/p/client.v8/builders/V8%20Fuzzer/builds/7930

The other three CLs are reverted to be able to revert the first.

BUG=v8:4798
LOG=n
TBR=verwaest@chromium.org
==========

to

==========
Revert "Speed up the LookupIterator"

This reverts commit 2608ecc7157a9d6d482709fc197b2fc0317d9d09.

Revert "Specialize helper methods in the LookupIterator by is_element."

This reverts commit 6eb483f8782e32d32f75d091fea4b50846210372.

Revert "Avoid SetPropertyInternal if the LookupIterator is NotFound"

This reverts commit ca5bd8d4a9d91336d770668dc569661e4fba5aa3.

Revert "Inline fast-bailout-checks for LookupIterator::UpdateProtector"

This reverts commit d98570a1eb7f017ca6ecc9d79236456f2691aee3.


This breaks layout tests with timeouts:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

It also seems to break jsfunfuzz:
https://build.chromium.org/p/client.v8/builders/V8%20Fuzzer/builds/7930

The other three CLs are reverted to be able to revert the first.

BUG=v8:4798
LOG=n
TBR=verwaest@chromium.org
==========

[commit-bot: I haz the power, 2016-03-03 08:10:50]

Committed patchset #1 (id:1)

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

Description was changed from

==========
Revert "Speed up the LookupIterator"

This reverts commit 2608ecc7157a9d6d482709fc197b2fc0317d9d09.

Revert "Specialize helper methods in the LookupIterator by is_element."

This reverts commit 6eb483f8782e32d32f75d091fea4b50846210372.

Revert "Avoid SetPropertyInternal if the LookupIterator is NotFound"

This reverts commit ca5bd8d4a9d91336d770668dc569661e4fba5aa3.

Revert "Inline fast-bailout-checks for LookupIterator::UpdateProtector"

This reverts commit d98570a1eb7f017ca6ecc9d79236456f2691aee3.


This breaks layout tests with timeouts:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

It also seems to break jsfunfuzz:
https://build.chromium.org/p/client.v8/builders/V8%20Fuzzer/builds/7930

The other three CLs are reverted to be able to revert the first.

BUG=v8:4798
LOG=n
TBR=verwaest@chromium.org
==========

to

==========
Revert "Speed up the LookupIterator"

This reverts commit 2608ecc7157a9d6d482709fc197b2fc0317d9d09.

Revert "Specialize helper methods in the LookupIterator by is_element."

This reverts commit 6eb483f8782e32d32f75d091fea4b50846210372.

Revert "Avoid SetPropertyInternal if the LookupIterator is NotFound"

This reverts commit ca5bd8d4a9d91336d770668dc569661e4fba5aa3.

Revert "Inline fast-bailout-checks for LookupIterator::UpdateProtector"

This reverts commit d98570a1eb7f017ca6ecc9d79236456f2691aee3.

This breaks layout tests with timeouts:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

It also seems to break jsfunfuzz:
https://build.chromium.org/p/client.v8/builders/V8%20Fuzzer/builds/7930

The other three CLs are reverted to be able to revert the first.

BUG=v8:4798
LOG=n
TBR=verwaest@chromium.org

Committed: https://crrev.com/70ac41a84a1dbb2025bb447a81582146ceb2a5ff
Cr-Commit-Position: refs/heads/master@{#34457}
==========

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

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