image_decoder service -> data_decoder service

image_decoder service -> data_decoder service

Establishes a more generic service definition for a thing to do
sandboxed data decoding. This is just a rename of the existing
image_decoder service.

Since there is nothing Chrome specific about this service, and there is
now some motivation to use sandboxed image decoding outside of Chrome,
this also moves the packaging of the service from chrome into content.

BUG=697557
TBR=tsepez@chromium.org

Review-Url: https://codereview.chromium.org/2774463002
Cr-Commit-Position: refs/heads/master@{#459631}
Committed: https://chromium.googlesource.com/chromium/src/+/e04370309e452bfdbad223228b908c7e31f2809c

[Ken Rockot(use gerrit already), 2017-03-22 22:29:44]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-22 22:30:35]

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

[Ken Rockot(use gerrit already), 2017-03-22 22:35:06]

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

[Ken Rockot(use gerrit already), 2017-03-22 22:35:07]

PTAL. Sorry for the size - most of the work is just a mechanical move.

One slightly unfortunate noteworthy thing here is that we had a few direct
consumers of the thin content::ImageDecoder and content::DecodeImage API
wrappers around blink image decoding. In order to avoid a dependency cycle
between content and data_decoder, I've moved that API into data_decoder and
introduced direct depenendencies onto the data_decoder implementation for
existing consumers.

In some cases - e.g. in an existing utility or render process that's already
sandboxed - I doubt we really want to have to connect to data_decoder
(potentially starting another process) just to decode data using some totally
service agnostic library code. So I think this is fine for now, modulo moving
the "internal" wrapper API to a more sensible non-content location outside of
the private data_decoder library.

[commit-bot: I haz the power, 2017-03-22 23:07:10]

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

[commit-bot: I haz the power, 2017-03-22 23:07:11]

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

[Ken Rockot(use gerrit already), 2017-03-23 23:30:02]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-23 23:30:38]

Dry run: 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-03-24 01:35:00]

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

[commit-bot: I haz the power, 2017-03-24 01:35:01]

Dry run: This issue passed the CQ dry run.

[Ken Rockot(use gerrit already), 2017-03-24 23:22:58]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-24 23:23:28]

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

[Ken Rockot(use gerrit already), 2017-03-24 23:44:06]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-24 23:44:22]

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

[jam, 2017-03-24 23:53:54]

lgtm

[Ken Rockot(use gerrit already), 2017-03-24 23:57:55]

The CQ bit was unchecked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2017-03-24 23:58:01]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2017-03-24 23:58:21]

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-03-25 00:09:00]

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

[commit-bot: I haz the power, 2017-03-25 00:09:01]

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

[Ken Rockot(use gerrit already), 2017-03-25 00:12:14]

Description was changed from

==========
image_decoder service -> data_decoder service

Establishes a more generic service definition for a thing to do
sandboxed data decoding. This is just a rename of the existing
image_decoder service.

Since there is nothing Chrome specific about this service, and there is
now some motivation to use sandboxed image decoding outside of Chrome,
this also moves the packaging of the service from chrome into content.

BUG=697557
==========

to

==========
image_decoder service -> data_decoder service

Establishes a more generic service definition for a thing to do
sandboxed data decoding. This is just a rename of the existing
image_decoder service.

Since there is nothing Chrome specific about this service, and there is
now some motivation to use sandboxed image decoding outside of Chrome,
this also moves the packaging of the service from chrome into content.

BUG=697557
TBR=tsepez@chromium.org
==========

[Ken Rockot(use gerrit already), 2017-03-25 00:13:29]

+TBR tsepez@, just shuffling some things around.

[Ken Rockot(use gerrit already), 2017-03-25 00:13:35]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2017-03-25 00:13: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, 2017-03-25 00:35:53]

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

[commit-bot: I haz the power, 2017-03-25 00:35:54]

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

[Ken Rockot(use gerrit already), 2017-03-25 01:31:46]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2017-03-25 01:32:22]

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-03-25 02:49:04]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490405506789170,
"parent_rev": "5adb8c047852670be6c9d42114f3a17d44b6c56e", "commit_rev":
"e04370309e452bfdbad223228b908c7e31f2809c"}

[commit-bot: I haz the power, 2017-03-25 02:50:44]

Description was changed from

==========
image_decoder service -> data_decoder service

Establishes a more generic service definition for a thing to do
sandboxed data decoding. This is just a rename of the existing
image_decoder service.

Since there is nothing Chrome specific about this service, and there is
now some motivation to use sandboxed image decoding outside of Chrome,
this also moves the packaging of the service from chrome into content.

BUG=697557
TBR=tsepez@chromium.org
==========

to

==========
image_decoder service -> data_decoder service

Establishes a more generic service definition for a thing to do
sandboxed data decoding. This is just a rename of the existing
image_decoder service.

Since there is nothing Chrome specific about this service, and there is
now some motivation to use sandboxed image decoding outside of Chrome,
this also moves the packaging of the service from chrome into content.

BUG=697557
TBR=tsepez@chromium.org

Review-Url: https://codereview.chromium.org/2774463002
Cr-Commit-Position: refs/heads/master@{#459631}
Committed:
https://chromium.googlesource.com/chromium/src/+/e04370309e452bfdbad223228b90...
==========

[commit-bot: I haz the power, 2017-03-25 02:50:46]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/e04370309e452bfdbad223228b90...