Revert of Make some functions that are hit during renderer startup available for inlining

Revert of Make some functions that are hit during renderer startup available for inlining (patchset #3 id:40001 of https://codereview.chromium.org/2950993002/ )

Reason for revert:
Blocks roll:
https://codereview.chromium.org/2954833002/

E.g.:
https://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_compile_dbg_ng/builds/449680
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_compile_dbg_ng/builds/324953

Please include those chromium trybots on reland. Maybe missing symbol export?

Original issue's description:
> Make some functions that are hit during renderer startup available for inlining
> 
> This is towards closing the perf gap between the MSVC build (which uses link-
> time optimization) and Clang (where LTO isn't ready on Windows yet). We did
> a study (see bug) to see which non-inlined functions are hit a lot during render
> start-up, and which would be inlined during LTO. This should benefit performance
> in all builds which currently don't use LTO (Android, Linux, Mac) as well as
> the Win/Clang build.
> 
> The binary size of chrome_child.dll increases by 2KB with this.
> 
> BUG=chromium:728324
> 
> Review-Url: https://codereview.chromium.org/2950993002
> Cr-Commit-Position: refs/heads/master@{#46191}
> Committed: https://chromium.googlesource.com/v8/v8/+/d00d52be1fce9c1bf5558c8b26bf984efd09e65b

TBR=jochen@chromium.org,mstarzinger@chromium.org,rmcilroy@chromium.org,vogelheim@chromium.org,marja@chromium.org,mlippautz@chromium.org,thakis@chromium.org,hans@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:728324
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2955793002
Cr-Commit-Position: refs/heads/master@{#46195}
Committed: https://chromium.googlesource.com/v8/v8/+/410a753ba2b0795e9d9b309c3a11ee5af6ab7b87

[Michael Achenbach, 2017-06-25 20:29:40]

The CQ bit was checked by machenbach@chromium.org

[Michael Achenbach, 2017-06-25 20:29:40]

Created Revert of Make some functions that are hit during renderer startup
available for inlining

[commit-bot: I haz the power, 2017-06-25 20:29:47]

CQ is trying da patch.

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

[Michael Achenbach, 2017-06-25 20:34:06]

Description was changed from

==========
Revert of Make some functions that are hit during renderer startup available for
inlining (patchset #3 id:40001 of https://codereview.chromium.org/2950993002/ )

Reason for revert:
Blocks roll:
https://codereview.chromium.org/2954833002/

E.g.:
https://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_com...
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

Please include those chromium trybots on reland. Maybe missing symbol export?

Original issue's description:
> Make some functions that are hit during renderer startup available for
inlining
> 
> This is towards closing the perf gap between the MSVC build (which uses link-
> time optimization) and Clang (where LTO isn't ready on Windows yet). We did
> a study (see bug) to see which non-inlined functions are hit a lot during
render
> start-up, and which would be inlined during LTO. This should benefit
performance
> in all builds which currently don't use LTO (Android, Linux, Mac) as well as
> the Win/Clang build.
> 
> The binary size of chrome_child.dll increases by 2KB with this.
> 
> BUG=chromium:728324
> 
> Review-Url: https://codereview.chromium.org/2950993002
> Cr-Commit-Position: refs/heads/master@{#46191}
> Committed:
https://chromium.googlesource.com/v8/v8/+/d00d52be1fce9c1bf5558c8b26bf984efd0...

TBR=jochen@chromium.org,mstarzinger@chromium.org,rmcilroy@chromium.org,vogelh...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:728324
==========

to

==========
Revert of Make some functions that are hit during renderer startup available for
inlining (patchset #3 id:40001 of https://codereview.chromium.org/2950993002/ )

Reason for revert:
Blocks roll:
https://codereview.chromium.org/2954833002/

E.g.:
https://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_com...
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

Please include those chromium trybots on reland. Maybe missing symbol export?

Original issue's description:
> Make some functions that are hit during renderer startup available for
inlining
> 
> This is towards closing the perf gap between the MSVC build (which uses link-
> time optimization) and Clang (where LTO isn't ready on Windows yet). We did
> a study (see bug) to see which non-inlined functions are hit a lot during
render
> start-up, and which would be inlined during LTO. This should benefit
performance
> in all builds which currently don't use LTO (Android, Linux, Mac) as well as
> the Win/Clang build.
> 
> The binary size of chrome_child.dll increases by 2KB with this.
> 
> BUG=chromium:728324
> 
> Review-Url: https://codereview.chromium.org/2950993002
> Cr-Commit-Position: refs/heads/master@{#46191}
> Committed:
https://chromium.googlesource.com/v8/v8/+/d00d52be1fce9c1bf5558c8b26bf984efd0...

TBR=jochen@chromium.org,mstarzinger@chromium.org,rmcilroy@chromium.org,vogelh...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:728324
NOTRY=true
NOPRESUBMIT=true
==========

[Michael Achenbach, 2017-06-25 20:34:13]

The CQ bit was unchecked by machenbach@chromium.org

[Michael Achenbach, 2017-06-25 20:34:17]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2017-06-25 20:34:26]

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, 2017-06-25 20:34:36]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498422857206450, "parent_rev":
"75f676f992a3abf7e6b1b655c431ac92429aebdf", "commit_rev":
"410a753ba2b0795e9d9b309c3a11ee5af6ab7b87"}

[commit-bot: I haz the power, 2017-06-25 20:34:47]

Description was changed from

==========
Revert of Make some functions that are hit during renderer startup available for
inlining (patchset #3 id:40001 of https://codereview.chromium.org/2950993002/ )

Reason for revert:
Blocks roll:
https://codereview.chromium.org/2954833002/

E.g.:
https://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_com...
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

Please include those chromium trybots on reland. Maybe missing symbol export?

Original issue's description:
> Make some functions that are hit during renderer startup available for
inlining
> 
> This is towards closing the perf gap between the MSVC build (which uses link-
> time optimization) and Clang (where LTO isn't ready on Windows yet). We did
> a study (see bug) to see which non-inlined functions are hit a lot during
render
> start-up, and which would be inlined during LTO. This should benefit
performance
> in all builds which currently don't use LTO (Android, Linux, Mac) as well as
> the Win/Clang build.
> 
> The binary size of chrome_child.dll increases by 2KB with this.
> 
> BUG=chromium:728324
> 
> Review-Url: https://codereview.chromium.org/2950993002
> Cr-Commit-Position: refs/heads/master@{#46191}
> Committed:
https://chromium.googlesource.com/v8/v8/+/d00d52be1fce9c1bf5558c8b26bf984efd0...

TBR=jochen@chromium.org,mstarzinger@chromium.org,rmcilroy@chromium.org,vogelh...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:728324
NOTRY=true
NOPRESUBMIT=true
==========

to

==========
Revert of Make some functions that are hit during renderer startup available for
inlining (patchset #3 id:40001 of https://codereview.chromium.org/2950993002/ )

Reason for revert:
Blocks roll:
https://codereview.chromium.org/2954833002/

E.g.:
https://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_com...
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

Please include those chromium trybots on reland. Maybe missing symbol export?

Original issue's description:
> Make some functions that are hit during renderer startup available for
inlining
> 
> This is towards closing the perf gap between the MSVC build (which uses link-
> time optimization) and Clang (where LTO isn't ready on Windows yet). We did
> a study (see bug) to see which non-inlined functions are hit a lot during
render
> start-up, and which would be inlined during LTO. This should benefit
performance
> in all builds which currently don't use LTO (Android, Linux, Mac) as well as
> the Win/Clang build.
> 
> The binary size of chrome_child.dll increases by 2KB with this.
> 
> BUG=chromium:728324
> 
> Review-Url: https://codereview.chromium.org/2950993002
> Cr-Commit-Position: refs/heads/master@{#46191}
> Committed:
https://chromium.googlesource.com/v8/v8/+/d00d52be1fce9c1bf5558c8b26bf984efd0...

TBR=jochen@chromium.org,mstarzinger@chromium.org,rmcilroy@chromium.org,vogelh...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:728324
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2955793002
Cr-Commit-Position: refs/heads/master@{#46195}
Committed:
https://chromium.googlesource.com/v8/v8/+/410a753ba2b0795e9d9b309c3a11ee5af6a...
==========

[commit-bot: I haz the power, 2017-06-25 20:34:48]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/410a753ba2b0795e9d9b309c3a11ee5af6a...

[hans, 2017-06-26 15:46:02]

Sorry for the breakage. Yes, the fix is simple; it's was a missing export:
https://codereview.chromium.org/2954353002/