Move common ImageFetcher component files to core/

Move common ImageFetcher component files to core/

Let's follow the directory structure of layered components.

TBR=mathp@chromium.org,sdefresne@chromium.org,jam@chromium.org
BUG=624761
Review-Url: https://codereview.chromium.org/2761303002
Cr-Commit-Position: refs/heads/master@{#458461}
Committed: https://chromium.googlesource.com/chromium/src/+/f4d088af9db25dfb569645655da829c28f84ee52

[mastiz, 2017-03-21 14:19:53]

mastiz@chromium.org changed reviewers:
+ treib@chromium.org

[mastiz, 2017-03-21 14:19:54]

treib@: let me know whether this is desirable and whether there's some automated
tool to achieve this. Right now I'm relying in things like:
git grep -l "include \"components/image_fetcher/" | xargs sed -i
's/components\/image_fetcher\//components\/image_fetcher\/core\//g'

[mastiz, 2017-03-21 14:22:45]

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

[commit-bot: I haz the power, 2017-03-21 14:23:04]

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

[Marc Treib, 2017-03-21 14:40:04]

lgtm

Yup, following the layered component structure definitely makes sense.

No, I don't know of a better way to update all the includes (but I tend to be
not the most knowledgeable person in these matters :D).

FWIW, all the include changes can be TBRed, no need to track down all the
owners.

https://codereview.chromium.org/2761303002/diff/20001/components/image_fetche...
File components/image_fetcher/core/image_fetcher_delegate.h (right):

https://codereview.chromium.org/2761303002/diff/20001/components/image_fetche...
components/image_fetcher/core/image_fetcher_delegate.h:27: const std::string&
data){};
I assume this is git cl formatted? Weird...

[mastiz, 2017-03-21 14:47:57]

Thanks.

https://codereview.chromium.org/2761303002/diff/20001/components/image_fetche...
File components/image_fetcher/core/image_fetcher_delegate.h (right):

https://codereview.chromium.org/2761303002/diff/20001/components/image_fetche...
components/image_fetcher/core/image_fetcher_delegate.h:27: const std::string&
data){};
On 2017/03/21 14:40:04, Marc Treib wrote:
> I assume this is git cl formatted? Weird...

Removed final semicolon which seems to cause this formatting.

[mastiz, 2017-03-21 16:32:35]

Description was changed from

==========
Move common ImageFetcher component files to core/

Let's follow the directory structure of layered components.

BUG=624761
==========

to

==========
Move common ImageFetcher component files to core/

Let's follow the directory structure of layered components.

TBR=mathp,sdefresne,jam
BUG=624761
==========

[mastiz, 2017-03-21 16:33:53]

Description was changed from

==========
Move common ImageFetcher component files to core/

Let's follow the directory structure of layered components.

TBR=mathp,sdefresne,jam
BUG=624761
==========

to

==========
Move common ImageFetcher component files to core/

Let's follow the directory structure of layered components.

TBR=mathp@chromium.org,sdefresne@chromium.org,jam@chromium.org
BUG=624761
==========

[mastiz, 2017-03-21 16:34:26]

mathp@, sdefresne@ ,jam@: TBR-ing due to trivial include updates.

[mastiz, 2017-03-21 16:35:26]

The CQ bit was checked by mastiz@chromium.org

[mastiz, 2017-03-21 16:35:26]

The patchset sent to the CQ was uploaded after l-g-t-m from treib@chromium.org
Link to the patchset: https://codereview.chromium.org/2761303002/#ps60001
(title: "Rebased.")

[commit-bot: I haz the power, 2017-03-21 16:35:50]

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-21 17:41:44]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490114126152200,
"parent_rev": "6e84ac1cc45e551a53b8381068cb91f0263f6b66", "commit_rev":
"f4d088af9db25dfb569645655da829c28f84ee52"}

[commit-bot: I haz the power, 2017-03-21 17:42:29]

Description was changed from

==========
Move common ImageFetcher component files to core/

Let's follow the directory structure of layered components.

TBR=mathp@chromium.org,sdefresne@chromium.org,jam@chromium.org
BUG=624761
==========

to

==========
Move common ImageFetcher component files to core/

Let's follow the directory structure of layered components.

TBR=mathp@chromium.org,sdefresne@chromium.org,jam@chromium.org
BUG=624761

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

[commit-bot: I haz the power, 2017-03-21 17:42:31]

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

[sdefresne, 2017-03-22 09:48:25]

sdefresne@chromium.org changed reviewers:
+ sdefresne@chromium.org

[sdefresne, 2017-03-22 09:48:26]

lgtm