Revert of [heap] Remove eager shortcut in JSFunction visitor.

Revert of [heap] Remove eager shortcut in JSFunction visitor. (patchset #1 id:1 of https://codereview.chromium.org/1476223002/ )

Reason for revert:
Still investigating bad canary.

Original issue's description:
> [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}

TBR=mstarzinger@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Committed: https://crrev.com/672b49119b857c4f96234b03e48b4b60de256969
Cr-Commit-Position: refs/heads/master@{#32463}

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

Created Revert of [heap] Remove eager shortcut in JSFunction visitor.

[commit-bot: I haz the power, 2015-12-01 13:30:39]

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

[commit-bot: I haz the power, 2015-12-01 13:32:05]

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

[commit-bot: I haz the power, 2015-12-01 13:32:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-01 13:33:00]

Description was changed from

==========
Revert of [heap] Remove eager shortcut in JSFunction visitor. (patchset #1 id:1
of https://codereview.chromium.org/1476223002/ )

Reason for revert:
Still investigating bad canary.

Original issue's description:
> [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}

TBR=mstarzinger@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Revert of [heap] Remove eager shortcut in JSFunction visitor. (patchset #1 id:1
of https://codereview.chromium.org/1476223002/ )

Reason for revert:
Still investigating bad canary.

Original issue's description:
> [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}

TBR=mstarzinger@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Committed: https://crrev.com/672b49119b857c4f96234b03e48b4b60de256969
Cr-Commit-Position: refs/heads/master@{#32463}
==========

[commit-bot: I haz the power, 2015-12-01 13:33:00]

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

[Hannes Payer (out of office), 2015-12-03 12:22:27]

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

The reason for reverting is: Suspect for crashing found, relanding for canary
coverage..