Sandbox html_viewer on Linux.

Sandbox html_viewer on Linux.

This creates a font_service which does the same role as
font_config_ipc_linux.cc does in content/. This lets us sandbox the
html_viewer, while still being able to communicate with fontconfig.

Also prewarms the ICU timezone data, along with doing all the
prewarming that's done in the core_services sandbox.

BUG=492524
NOPRESUBMIT=true

Committed: https://crrev.com/4196dfefc131b0a77a3e02e917b852dba8faf445
Cr-Commit-Position: refs/heads/master@{#342281}

[Elliot Glaysher, 2015-08-06 20:51:06]

erg@chromium.org changed reviewers:
+ jam@chromium.org

[jam, 2015-08-06 21:34:38]

lgtm

https://codereview.chromium.org/1274743004/diff/80001/components/font_service...
File components/font_service/public/cpp/font_loader.h (right):

https://codereview.chromium.org/1274743004/diff/80001/components/font_service...
components/font_service/public/cpp/font_loader.h:41: FontIdentity*
outFontIdentifier,
nit: convention is to rename parameters to use unix_hacker style, even if the
overridden method used CamelCase

also in other files

https://codereview.chromium.org/1274743004/diff/80001/components/font_service...
File components/font_service/public/cpp/font_service_thread.h (right):

https://codereview.chromium.org/1274743004/diff/80001/components/font_service...
components/font_service/public/cpp/font_service_thread.h:33: // to the
FontServiceThread, and
nit:

https://codereview.chromium.org/1274743004/diff/80001/components/font_service...
components/font_service/public/cpp/font_service_thread.h:47: // calls this
method, this method calls the mojo interface, and
nit:

[Elliot Glaysher, 2015-08-06 22:58:17]

(adding NOPRESUBMIT because of the copied ScopedAllowIO code.)

[Elliot Glaysher, 2015-08-06 22:58:23]

The CQ bit was checked by erg@chromium.org

[Elliot Glaysher, 2015-08-06 22:58:23]

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/1274743004/#ps100001
(title: "Style stuff.")

[commit-bot: I haz the power, 2015-08-06 22:58:53]

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

[commit-bot: I haz the power, 2015-08-06 23:57:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-06 23:57:57]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Elliot Glaysher, 2015-08-07 00:24:42]

The CQ bit was checked by erg@chromium.org

[commit-bot: I haz the power, 2015-08-07 00:25:16]

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

[commit-bot: I haz the power, 2015-08-07 03:06:51]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-08-07 03:07:38]

Patchset 6 (id:??) landed as
https://crrev.com/4196dfefc131b0a77a3e02e917b852dba8faf445
Cr-Commit-Position: refs/heads/master@{#342281}

[hiroshige (private), 2015-08-07 04:10:53]

This CL causes compile failures on Linux buildbots due to conflict with
renaming of mojo headers in https://codereview.chromium.org/1262173005/:
../../components/font_service/public/cpp/font_service_thread.cc:11:10: fatal
error: 'mojo/common/message_pump_mojo.h' file not found
#include "mojo/common/message_pump_mojo.h"

[Nico, 2015-08-07 04:49:34]

A revert of this CL (patchset #6 id:100001) has been created in
https://codereview.chromium.org/1276163002/ by thakis@chromium.org.

The reason for reverting is: Doesn't build on the Linux GN bot:
http://build.chromium.org/p/chromium.linux/builders/Linux%20GN/builds/30188/s....