Loading: split MockResourceClients into multiple files type by type

Loading: split MockResourceClients into multiple files type by type

This patch splits MockResourceClients into MockResourceClient and
MockImageResourceClient, because MemoryCacheTest and ResourceFetcherTest
just need MockResourceClient and do not want to depend on
MockImageResourceClient that has a dependency to ImageResource.

BUG=655920
Committed: https://crrev.com/e3d284d7654727b708bddce7b28bb205a6076ca1
Cr-Commit-Position: refs/heads/master@{#438109}

[Takashi Toyoshima, 2016-12-07 12:31:44]

Description was changed from

==========
Loading: split MockResourceClients into multiple files type by type

This patch split MockResourceClients into MockResourceClient and
MockImageResourceClient, because MemoryCacheTest and ResourceFetcherTest
just need MockResourceClient and do not want to depend on
MockImageResourceClient that has a dependency to ImageResource.

BUG=655920
==========

to

==========
Loading: split MockResourceClients into multiple files type by type

This patch split MockResourceClients into MockResourceClient and
MockImageResourceClient, because MemoryCacheTest and ResourceFetcherTest
just need MockResourceClient and do not want to depend on
MockImageResourceClient that has a dependency to ImageResource.

BUG=655920
==========

[Takashi Toyoshima, 2016-12-07 12:31:44]

toyoshim@chromium.org changed reviewers:
+ hiroshige@chromium.org, yhirano@chromium.org

[Takashi Toyoshima, 2016-12-07 12:31:55]

ptal

[hiroshige, 2016-12-12 14:36:27]

lgtm.

[yhirano, 2016-12-13 03:49:42]

lgtm

> This patch split
splits

[Takashi Toyoshima, 2016-12-13 07:45:19]

Description was changed from

==========
Loading: split MockResourceClients into multiple files type by type

This patch split MockResourceClients into MockResourceClient and
MockImageResourceClient, because MemoryCacheTest and ResourceFetcherTest
just need MockResourceClient and do not want to depend on
MockImageResourceClient that has a dependency to ImageResource.

BUG=655920
==========

to

==========
Loading: split MockResourceClients into multiple files type by type

This patch splits MockResourceClients into MockResourceClient and
MockImageResourceClient, because MemoryCacheTest and ResourceFetcherTest
just need MockResourceClient and do not want to depend on
MockImageResourceClient that has a dependency to ImageResource.

BUG=655920
==========

[Takashi Toyoshima, 2016-12-13 07:45:30]

toyoshim@chromium.org changed reviewers:
+ tkent@chromium.org

[Takashi Toyoshima, 2016-12-13 07:45:53]

toyoshim@chromium.org changed reviewers:
- tkent@chromium.org

[Takashi Toyoshima, 2016-12-13 07:46:11]

toyoshim@chromium.org changed reviewers:
+ kouhei@chromium.org

[Takashi Toyoshima, 2016-12-13 07:46:25]

+kouhei for BUIDL.gn

[Takashi Toyoshima, 2016-12-13 07:47:10]

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

[commit-bot: I haz the power, 2016-12-13 07:47:24]

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, 2016-12-13 09:15:26]

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

[commit-bot: I haz the power, 2016-12-13 09:15:27]

Dry run: This issue passed the CQ dry run.

[kouhei (in TOK), 2016-12-13 09:17:41]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2016-12-13 09:17:41]

lgtm

rs

[kouhei (in TOK), 2016-12-13 09:17:42]

The patchset sent to the CQ was uploaded after l-g-t-m from
hiroshige@chromium.org, yhirano@chromium.org
Link to the patchset: https://codereview.chromium.org/2559703002/#ps20001
(title: "[rebase to trunk]")

[kouhei (in TOK), 2016-12-13 09:17:49]

The CQ bit was unchecked by kouhei@chromium.org

[Takashi Toyoshima, 2016-12-13 09:20:30]

The CQ bit was checked by toyoshim@chromium.org

[commit-bot: I haz the power, 2016-12-13 09:20:43]

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-12-13 09:24:44]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481620830748100,
"parent_rev": "9c39bf509a9c6459f55b7d9200c29ab56236cd5c", "commit_rev":
"2d7abb78d11d16d614265216e8bab6b3fde28672"}

[commit-bot: I haz the power, 2016-12-13 09:25:12]

Description was changed from

==========
Loading: split MockResourceClients into multiple files type by type

This patch splits MockResourceClients into MockResourceClient and
MockImageResourceClient, because MemoryCacheTest and ResourceFetcherTest
just need MockResourceClient and do not want to depend on
MockImageResourceClient that has a dependency to ImageResource.

BUG=655920
==========

to

==========
Loading: split MockResourceClients into multiple files type by type

This patch splits MockResourceClients into MockResourceClient and
MockImageResourceClient, because MemoryCacheTest and ResourceFetcherTest
just need MockResourceClient and do not want to depend on
MockImageResourceClient that has a dependency to ImageResource.

BUG=655920

Review-Url: https://codereview.chromium.org/2559703002
==========

[commit-bot: I haz the power, 2016-12-13 09:25:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-13 09:29:19]

Description was changed from

==========
Loading: split MockResourceClients into multiple files type by type

This patch splits MockResourceClients into MockResourceClient and
MockImageResourceClient, because MemoryCacheTest and ResourceFetcherTest
just need MockResourceClient and do not want to depend on
MockImageResourceClient that has a dependency to ImageResource.

BUG=655920

Review-Url: https://codereview.chromium.org/2559703002
==========

to

==========
Loading: split MockResourceClients into multiple files type by type

This patch splits MockResourceClients into MockResourceClient and
MockImageResourceClient, because MemoryCacheTest and ResourceFetcherTest
just need MockResourceClient and do not want to depend on
MockImageResourceClient that has a dependency to ImageResource.

BUG=655920

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

[commit-bot: I haz the power, 2016-12-13 09:29:20]

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