Revert of Refactor SharedFunctionInfo::IsBuiltin.

Revert of Refactor SharedFunctionInfo::IsBuiltin. (patchset #1 id:1 of https://codereview.chromium.org/2505853003/ )

Reason for revert:
Breaks layout tests:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/11394

Original issue's description:
> Refactor SharedFunctionInfo::IsBuiltin.
>
> This method is a slight misnomer. What we actually want to know is
> whether the function was defined in a user-provided script.
>
> Also remove redundant Script::hide_source flag.
>
> R=bmeurer@chromium.org, ulan@chromium.org

TBR=bmeurer@chromium.org,ulan@chromium.org,yangguo@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Committed: https://crrev.com/1160e5edccdecb282c9fd8ad36acbfb4400b5243
Cr-Commit-Position: refs/heads/master@{#41050}

[Michael Achenbach, 2016-11-16 19:47:43]

The CQ bit was checked by machenbach@chromium.org

[Michael Achenbach, 2016-11-16 19:47:44]

Created Revert of Refactor SharedFunctionInfo::IsBuiltin.

[commit-bot: I haz the power, 2016-11-16 19:47:49]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-16 19:48:03]

Description was changed from

==========
Revert of Refactor SharedFunctionInfo::IsBuiltin. (patchset #1 id:1 of
https://codereview.chromium.org/2505853003/ )

Reason for revert:
Breaks layout tests:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

Original issue's description:
> Refactor SharedFunctionInfo::IsBuiltin.
> 
> This method is a slight misnomer. What we actually want to know is
> whether the function was defined in a user-provided script.
> 
> Also remove redundant Script::hide_source flag.
> 
> R=bmeurer@chromium.org, ulan@chromium.org

TBR=bmeurer@chromium.org,ulan@chromium.org,yangguo@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Revert of Refactor SharedFunctionInfo::IsBuiltin. (patchset #1 id:1 of
https://codereview.chromium.org/2505853003/ )

Reason for revert:
Breaks layout tests:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

Original issue's description:
> Refactor SharedFunctionInfo::IsBuiltin.
> 
> This method is a slight misnomer. What we actually want to know is
> whether the function was defined in a user-provided script.
> 
> Also remove redundant Script::hide_source flag.
> 
> R=bmeurer@chromium.org, ulan@chromium.org

TBR=bmeurer@chromium.org,ulan@chromium.org,yangguo@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

[commit-bot: I haz the power, 2016-11-16 19:48:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 22:37:00]

Description was changed from

==========
Revert of Refactor SharedFunctionInfo::IsBuiltin. (patchset #1 id:1 of
https://codereview.chromium.org/2505853003/ )

Reason for revert:
Breaks layout tests:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

Original issue's description:
> Refactor SharedFunctionInfo::IsBuiltin.
> 
> This method is a slight misnomer. What we actually want to know is
> whether the function was defined in a user-provided script.
> 
> Also remove redundant Script::hide_source flag.
> 
> R=bmeurer@chromium.org, ulan@chromium.org

TBR=bmeurer@chromium.org,ulan@chromium.org,yangguo@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Revert of Refactor SharedFunctionInfo::IsBuiltin. (patchset #1 id:1 of
https://codereview.chromium.org/2505853003/ )

Reason for revert:
Breaks layout tests:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

Original issue's description:
> Refactor SharedFunctionInfo::IsBuiltin.
>
> This method is a slight misnomer. What we actually want to know is
> whether the function was defined in a user-provided script.
>
> Also remove redundant Script::hide_source flag.
>
> R=bmeurer@chromium.org, ulan@chromium.org

TBR=bmeurer@chromium.org,ulan@chromium.org,yangguo@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Committed: https://crrev.com/1160e5edccdecb282c9fd8ad36acbfb4400b5243
Cr-Commit-Position: refs/heads/master@{#41050}
==========

[commit-bot: I haz the power, 2016-11-17 22:37:01]

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