Add com.apple.fonts to the sandbox profiles for Sierra

Add com.apple.fonts to the sandbox profiles for Sierra

It was done for WebKit in http://wkrev.com/192805

Without it, the renderer process can't talk to the font server on macOS
10.12 Sierra and spawns an entire font server in each renderer process.

This CL reduces the memory footprint of each renderer process by ~600MB
on macOS 10.12 Beta (16A201w).

BUG=619981
Committed: https://crrev.com/56fd33cb2374b335e1c309c2af2940ad469ccee3
Cr-Commit-Position: refs/heads/master@{#402523}

[tapted, 2016-06-17 02:52:15]

Description was changed from

==========
Add com.apple.fonts to the sandbox profiles for Sierra

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

to

==========
Add com.apple.fonts to the sandbox profiles for Sierra

It was done for WebKit in http://wkrev.com/192805

Without it, the renderer process can't talk to the font server on macOS
10.12 Sierra and spawns an entire font server in each renderer process.

This reduces the memory footprint of each renderer process by ~600MB on
macOS 10.12 Beta (16A201w).

BUG=619981
==========

[tapted, 2016-06-17 02:52:15]

tapted@chromium.org changed reviewers:
+ rsesek@chromium.org

[tapted, 2016-06-17 02:52:40]

Description was changed from

==========
Add com.apple.fonts to the sandbox profiles for Sierra

It was done for WebKit in http://wkrev.com/192805

Without it, the renderer process can't talk to the font server on macOS
10.12 Sierra and spawns an entire font server in each renderer process.

This reduces the memory footprint of each renderer process by ~600MB on
macOS 10.12 Beta (16A201w).

BUG=619981
==========

to

==========
Add com.apple.fonts to the sandbox profiles for Sierra

It was done for WebKit in http://wkrev.com/192805

Without it, the renderer process can't talk to the font server on macOS
10.12 Sierra and spawns an entire font server in each renderer process.

This CL reduces the memory footprint of each renderer process by ~600MB
on macOS 10.12 Beta (16A201w).

BUG=619981
==========

[tapted, 2016-06-17 02:53:22]

Hi Robert, please take a look

[Robert Sesek, 2016-06-17 20:51:15]

rsesek@chromium.org changed reviewers:
+ kerrnel@chromium.org

[Robert Sesek, 2016-06-17 20:51:16]

Ack'ing this CL. But before approving this either Greg or I need to look at what
sort of attack surface this is going to open up in the renderer process.

[Greg K, 2016-06-28 18:16:26]

On 2016/06/17 20:51:16, Robert Sesek wrote:
> Ack'ing this CL. But before approving this either Greg or I need to look at
what
> sort of attack surface this is going to open up in the renderer process.

I dug into it, and we think that all the messages to the new server look
reasonable. Thanks for helping Robert.

LGTM

[Robert Sesek, 2016-06-28 18:17:33]

OWNERS stamp LGTM

[Robert Sesek, 2016-06-28 18:17:44]

On 2016/06/28 18:16:26, Greg Kerr wrote:
> On 2016/06/17 20:51:16, Robert Sesek wrote:
> > Ack'ing this CL. But before approving this either Greg or I need to look at
> what
> > sort of attack surface this is going to open up in the renderer process.
> 
> I dug into it, and we think that all the messages to the new server look
> reasonable. Thanks for helping Robert.
> 
> LGTM

Thanks, Greg!

[Robert Sesek, 2016-06-28 18:31:45]

The CQ bit was checked by rsesek@chromium.org

[commit-bot: I haz the power, 2016-06-28 18:34:18]

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-06-28 20:22:08]

Description was changed from

==========
Add com.apple.fonts to the sandbox profiles for Sierra

It was done for WebKit in http://wkrev.com/192805

Without it, the renderer process can't talk to the font server on macOS
10.12 Sierra and spawns an entire font server in each renderer process.

This CL reduces the memory footprint of each renderer process by ~600MB
on macOS 10.12 Beta (16A201w).

BUG=619981
==========

to

==========
Add com.apple.fonts to the sandbox profiles for Sierra

It was done for WebKit in http://wkrev.com/192805

Without it, the renderer process can't talk to the font server on macOS
10.12 Sierra and spawns an entire font server in each renderer process.

This CL reduces the memory footprint of each renderer process by ~600MB
on macOS 10.12 Beta (16A201w).

BUG=619981
==========

[commit-bot: I haz the power, 2016-06-28 20:22:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-28 20:25:44]

Description was changed from

==========
Add com.apple.fonts to the sandbox profiles for Sierra

It was done for WebKit in http://wkrev.com/192805

Without it, the renderer process can't talk to the font server on macOS
10.12 Sierra and spawns an entire font server in each renderer process.

This CL reduces the memory footprint of each renderer process by ~600MB
on macOS 10.12 Beta (16A201w).

BUG=619981
==========

to

==========
Add com.apple.fonts to the sandbox profiles for Sierra

It was done for WebKit in http://wkrev.com/192805

Without it, the renderer process can't talk to the font server on macOS
10.12 Sierra and spawns an entire font server in each renderer process.

This CL reduces the memory footprint of each renderer process by ~600MB
on macOS 10.12 Beta (16A201w).

BUG=619981

Committed: https://crrev.com/56fd33cb2374b335e1c309c2af2940ad469ccee3
Cr-Commit-Position: refs/heads/master@{#402523}
==========

[commit-bot: I haz the power, 2016-06-28 20:25:45]

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