[heap] Remove eager shortcut in JSFunction visitor.

[heap] Remove eager shortcut in JSFunction visitor.

This removes an optimization in the static JSFunction visitor that
eagerly marked through to the SharedFunctionInfo for code flushing
candidates. This causes all processing in VisitJSFunction to be
side-stepped and hence might cause leaks.

R=hpayer@chromium.org

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

[Hannes Payer (out of office), 2015-11-26 09:53:46]

lgtm

[Michael Starzinger, 2015-11-26 11:34:42]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2015-11-26 11:34:47]

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

[commit-bot: I haz the power, 2015-11-26 11:36:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-26 11:36:39]

Description was changed from

==========
[heap] Remove eager shortcut in JSFunction visitor.

This removes an optimization in the static JSFunction visitor that
eagerly marked through to the SharedFunctionInfo for code flushing
candidates. This causes all processing in VisitJSFunction to be
side-stepped and hence might cause leaks.

R=hpayer@chromium.org
==========

to

==========
[heap] Remove eager shortcut in JSFunction visitor.

This removes an optimization in the static JSFunction visitor that
eagerly marked through to the SharedFunctionInfo for code flushing
candidates. This causes all processing in VisitJSFunction to be
side-stepped and hence might cause leaks.

R=hpayer@chromium.org

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

[commit-bot: I haz the power, 2015-11-26 11:36:39]

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

[Hannes Payer (out of office), 2015-12-01 13:30:30]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1488063002/ by hpayer@chromium.org.

The reason for reverting is: Still investigating bad canary..