Loading: use MockFetchContext in ImageResourceTest

Loading: use MockFetchContext in ImageResourceTest

ImageResourceTest still has a FetchContext subclass for testing.
But MockFetchContext can replace it.

BUG=655920
Review-Url: https://codereview.chromium.org/2692203008
Cr-Commit-Position: refs/heads/master@{#451221}
Committed: https://chromium.googlesource.com/chromium/src/+/96ce5a090b950b68977e03b4c8f5e10d5ab8949d

[Takashi Toyoshima, 2017-02-16 12:43:45]

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

[commit-bot: I haz the power, 2017-02-16 12:44:00]

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

[Takashi Toyoshima, 2017-02-16 12:44:27]

Description was changed from

==========
Loading: use MockFetchContext in ImageResourceTest

ImageResourceTest still have a FetchContext subclass for testing.
But MockFetchContext can replace it.

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

to

==========
Loading: use MockFetchContext in ImageResourceTest

ImageResourceTest still have a FetchContext subclass for testing.
But MockFetchContext can replace it.

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

[Takashi Toyoshima, 2017-02-16 12:44:27]

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

[Takashi Toyoshima, 2017-02-16 12:45:50]

ptal

[commit-bot: I haz the power, 2017-02-16 13:39:26]

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

[commit-bot: I haz the power, 2017-02-16 13:39:27]

Dry run: This issue passed the CQ dry run.

[yhirano, 2017-02-17 03:48:21]

lgtm.

> ImageResourceTest still have
has

[Takashi Toyoshima, 2017-02-17 04:32:25]

Description was changed from

==========
Loading: use MockFetchContext in ImageResourceTest

ImageResourceTest still have a FetchContext subclass for testing.
But MockFetchContext can replace it.

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

to

==========
Loading: use MockFetchContext in ImageResourceTest

ImageResourceTest still has a FetchContext subclass for testing.
But MockFetchContext can replace it.

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

[Takashi Toyoshima, 2017-02-17 04:32:36]

The CQ bit was checked by toyoshim@chromium.org

[commit-bot: I haz the power, 2017-02-17 04:33:40]

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-02-17 05:03:46]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487305956335460, "parent_rev":
"fa44a170953dea4680dc68f1eb0e145b97307958", "commit_rev":
"96ce5a090b950b68977e03b4c8f5e10d5ab8949d"}

[commit-bot: I haz the power, 2017-02-17 05:05:19]

Description was changed from

==========
Loading: use MockFetchContext in ImageResourceTest

ImageResourceTest still has a FetchContext subclass for testing.
But MockFetchContext can replace it.

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

to

==========
Loading: use MockFetchContext in ImageResourceTest

ImageResourceTest still has a FetchContext subclass for testing.
But MockFetchContext can replace it.

BUG=655920

Review-Url: https://codereview.chromium.org/2692203008
Cr-Commit-Position: refs/heads/master@{#451221}
Committed:
https://chromium.googlesource.com/chromium/src/+/96ce5a090b950b68977e03b4c8f5...
==========

[commit-bot: I haz the power, 2017-02-17 05:05:21]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/96ce5a090b950b68977e03b4c8f5...