Revert of content_shell: Remove deps for resources only needed at runtime.

Revert of content_shell: Remove deps for resources only needed at runtime. (patchset #1 id:1 of https://codereview.chromium.org/2469073004/ )

Reason for revert:
Broke Mac builds of content shell. See Robert Sesek's comments.

Original issue's description:
> content_shell: Remove deps for resources only needed at runtime.
>
> https://chromium.googlesource.com/chromium/src/+/master/tools/gn/docs/reference.md#data_deps_Non_linked-dependencies
> > data_deps are non-linked runtime dependencies. They are built and available
> > at runtime.
>
> The .pak and font resources are not linked but rather just needed at
> runtime.
>
> Split out of https://codereview.chromium.org/2460503002/
>
> BUG=524758
>
> Committed: https://crrev.com/3d9a6345c105cd59cb62f6c5aa9649ac4560b0b0
> Cr-Commit-Position: refs/heads/master@{#429517}

TBR=jochen@chromium.org,dpranke@google.com,dpranke@chromium.org,rsesek@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=524758
Committed: https://crrev.com/fa69073c14cec82dac0bed3a16583e63f84e001b
Cr-Commit-Position: refs/heads/master@{#434571}

[mithro, 2016-11-23 23:21:44]

The CQ bit was checked by tansell@chromium.org

[mithro, 2016-11-23 23:21:45]

Created Revert of content_shell: Remove deps for resources only needed at
runtime.

[commit-bot: I haz the power, 2016-11-23 23:22:48]

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

[Dirk Pranke, 2016-11-24 00:22:23]

lgtm

[mithro, 2016-11-24 00:52:04]

On 2016/11/24 00:22:23, Dirk Pranke wrote:
> lgtm

I'm reverting this patch for now.

I spent a bunch of time yesterday trying to figure out the correct solution for
getting Mac's .app to include the files, but as I don't have a Mac and running
on the bots is slow, I wasn't able to figure out the correct solution for
getting Mac to include the files.

Tim 'mithro' Ansell

[commit-bot: I haz the power, 2016-11-24 01:25:45]

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

[commit-bot: I haz the power, 2016-11-24 01:25:46]

Try jobs failed on following builders:
  blimp_linux_dbg on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)

[mithro, 2016-11-24 03:02:18]

The CQ bit was checked by tansell@chromium.org

[commit-bot: I haz the power, 2016-11-24 03:02:57]

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-24 05:04:00]

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

[commit-bot: I haz the power, 2016-11-24 05:04:01]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[mithro, 2016-11-24 06:12:33]

The CQ bit was checked by tansell@chromium.org

[commit-bot: I haz the power, 2016-11-24 06:13:00]

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-24 08:14:27]

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

[commit-bot: I haz the power, 2016-11-24 08:14:28]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[jochen (gone - plz use gerrit), 2016-11-24 11:56:29]

lgtm

[mithro, 2016-11-25 00:55:33]

The CQ bit was checked by tansell@chromium.org

[commit-bot: I haz the power, 2016-11-25 00:55:42]

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-25 01:35:46]

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

[commit-bot: I haz the power, 2016-11-25 01:35:48]

Try jobs failed on following builders:
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[mithro, 2016-11-26 04:16:46]

The CQ bit was checked by tansell@chromium.org

[commit-bot: I haz the power, 2016-11-26 04:17:02]

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-26 07:06:35]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480133806857510, "parent_rev":
"8dca4f48031610b1364a02ea5c7e03d16ec430d5", "commit_rev":
"db2f67995fa696d7923a3bf4d4187e7439db4d45"}

[commit-bot: I haz the power, 2016-11-26 07:07:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-26 07:09:48]

Description was changed from

==========
Revert of content_shell: Remove deps for resources only needed at runtime.
(patchset #1 id:1 of https://codereview.chromium.org/2469073004/ )

Reason for revert:
Broke Mac builds of content shell. See Robert Sesek's comments.

Original issue's description:
> content_shell: Remove deps for resources only needed at runtime.
> 
>
https://chromium.googlesource.com/chromium/src/+/master/tools/gn/docs/referen...
> > data_deps are non-linked runtime dependencies. They are built and available
> > at runtime.
> 
> The .pak and font resources are not linked but rather just needed at
> runtime.
> 
> Split out of https://codereview.chromium.org/2460503002/
> 
> BUG=524758
> 
> Committed: https://crrev.com/3d9a6345c105cd59cb62f6c5aa9649ac4560b0b0
> Cr-Commit-Position: refs/heads/master@{#429517}

TBR=jochen@chromium.org,dpranke@google.com,dpranke@chromium.org,rsesek@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=524758
==========

to

==========
Revert of content_shell: Remove deps for resources only needed at runtime.
(patchset #1 id:1 of https://codereview.chromium.org/2469073004/ )

Reason for revert:
Broke Mac builds of content shell. See Robert Sesek's comments.

Original issue's description:
> content_shell: Remove deps for resources only needed at runtime.
>
>
https://chromium.googlesource.com/chromium/src/+/master/tools/gn/docs/referen...
> > data_deps are non-linked runtime dependencies. They are built and available
> > at runtime.
>
> The .pak and font resources are not linked but rather just needed at
> runtime.
>
> Split out of https://codereview.chromium.org/2460503002/
>
> BUG=524758
>
> Committed: https://crrev.com/3d9a6345c105cd59cb62f6c5aa9649ac4560b0b0
> Cr-Commit-Position: refs/heads/master@{#429517}

TBR=jochen@chromium.org,dpranke@google.com,dpranke@chromium.org,rsesek@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=524758

Committed: https://crrev.com/fa69073c14cec82dac0bed3a16583e63f84e001b
Cr-Commit-Position: refs/heads/master@{#434571}
==========

[commit-bot: I haz the power, 2016-11-26 07:09:49]

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