[ImageResource 1a] Change StyleFetchedImage from ResourceClient to ImageResourceObserver

Change StyleFetchedImage from ResourceClient to ImageResourceObserver

This CL is a part of moving ResourceClient subclasses used for ImageResource
to ImageResourceObserver, in order to split ImageResource into Image-part and
Resource-part.

BUG=667641
Committed: https://crrev.com/37e0c8e928dfad9c455b5f2145a5e3e25e1350ca
Cr-Commit-Position: refs/heads/master@{#435269}

[hiroshige, 2016-11-09 07:15:40]

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

[commit-bot: I haz the power, 2016-11-09 07:15:47]

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

[hiroshige, 2016-11-09 07:20:14]

Description was changed from

==========
Change StyleFetchedImage from ResourceClient into ImageResourceObserver

BUG=
==========

to

==========
Change StyleFetchedImage from ResourceClient into ImageResourceObserver

BUG=
==========

[commit-bot: I haz the power, 2016-11-09 10:55:51]

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

[commit-bot: I haz the power, 2016-11-09 10:55:51]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-11-09 11:37:47]

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

[commit-bot: I haz the power, 2016-11-09 11:38:01]

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-11-09 14:12:16]

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

[commit-bot: I haz the power, 2016-11-09 14:12:17]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-11-22 06:53:08]

Description was changed from

==========
Change StyleFetchedImage from ResourceClient into ImageResourceObserver

BUG=
==========

to

==========
Change StyleFetchedImage from ResourceClient into ImageResourceObserver

BUG=667641
==========

[hiroshige, 2016-11-22 07:58:02]

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

[commit-bot: I haz the power, 2016-11-22 07:58:16]

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-11-22 09:28:34]

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

[commit-bot: I haz the power, 2016-11-22 09:28:34]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-11-22 11:55:46]

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

[commit-bot: I haz the power, 2016-11-22 11:55:56]

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

[hiroshige, 2016-11-22 11:59:27]

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

[commit-bot: I haz the power, 2016-11-22 11:59:42]

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

[hiroshige, 2016-11-22 12:05:34]

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

[commit-bot: I haz the power, 2016-11-22 12:05:41]

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-11-22 13:30:06]

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

[commit-bot: I haz the power, 2016-11-22 13:30:07]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-11-22 18:49:16]

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

[commit-bot: I haz the power, 2016-11-22 18:49:46]

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-11-22 19:19:29]

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

[commit-bot: I haz the power, 2016-11-22 19:19:30]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[hiroshige, 2016-11-25 04:58:29]

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

[hiroshige, 2016-11-25 10:10:43]

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

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

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-11-25 11:52:41]

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

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

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-11-28 06:23:39]

Description was changed from

==========
Change StyleFetchedImage from ResourceClient into ImageResourceObserver

BUG=667641
==========

to

==========
Change StyleFetchedImage from ResourceClient to ImageResourceObserver

This CL is a part of moving ResourceClient subclasses used for ImageResource
to ImageResourceObserver, in order to split ImageResource into Image-part and
Resource-part.

BUG=667641
==========

[hiroshige, 2016-11-28 06:24:09]

PTAL.

[hiroshige, 2016-11-29 07:09:30]

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

[commit-bot: I haz the power, 2016-11-29 07:09:57]

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

[yhirano, 2016-11-29 08:26:59]

lgtm

https://codereview.chromium.org/2486073004/diff/140001/third_party/WebKit/Sou...
File third_party/WebKit/Source/core/style/StyleFetchedImage.h (right):

https://codereview.chromium.org/2486073004/diff/140001/third_party/WebKit/Sou...
third_party/WebKit/Source/core/style/StyleFetchedImage.h:28: #include
"core/fetch/ResourceClient.h"
Not needed?

[commit-bot: I haz the power, 2016-11-29 08:37:48]

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

[commit-bot: I haz the power, 2016-11-29 08:37:49]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-11-29 09:13:45]

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

[hiroshige, 2016-11-29 09:13:46]

+kouhei, PTAL as core/ OWNER.

[hiroshige, 2016-11-29 09:16:41]

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

[commit-bot: I haz the power, 2016-11-29 09:17:02]

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

[hiroshige, 2016-11-29 09:18:01]

https://codereview.chromium.org/2486073004/diff/140001/third_party/WebKit/Sou...
File third_party/WebKit/Source/core/style/StyleFetchedImage.h (right):

https://codereview.chromium.org/2486073004/diff/140001/third_party/WebKit/Sou...
third_party/WebKit/Source/core/style/StyleFetchedImage.h:28: #include
"core/fetch/ResourceClient.h"
On 2016/11/29 08:26:59, yhirano wrote:
> Not needed?

Done.

[kouhei (in TOK), 2016-11-29 09:18:16]

lgtm

[commit-bot: I haz the power, 2016-11-29 10:33:04]

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

[commit-bot: I haz the power, 2016-11-29 10:33:05]

Dry run: This issue passed the CQ dry run.

[kouhei (in TOK), 2016-11-30 01:39:03]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2016-11-30 01:39:03]

The patchset sent to the CQ was uploaded after l-g-t-m from yhirano@chromium.org
Link to the patchset: https://codereview.chromium.org/2486073004/#ps180001
(title: "Rebase")

[commit-bot: I haz the power, 2016-11-30 01:40:21]

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

[commit-bot: I haz the power, 2016-11-30 01:40:23]

This CL has an open dependency (Issue 2468883003 Patch 220001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[hiroshige, 2016-11-30 05:56:54]

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

[commit-bot: I haz the power, 2016-11-30 05:57:42]

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-11-30 09:21:15]

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

[commit-bot: I haz the power, 2016-11-30 09:21:16]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[hiroshige, 2016-11-30 14:11:42]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2016-11-30 14:11:43]

The patchset sent to the CQ was uploaded after l-g-t-m from
yhirano@chromium.org, kouhei@chromium.org
Link to the patchset: https://codereview.chromium.org/2486073004/#ps200001
(title: "Rebase")

[commit-bot: I haz the power, 2016-11-30 14:12:01]

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-30 15:51:04]

CQ is committing da patch.

Bot data: {"patchset_id": 200001, "attempt_start_ts": 1480515102799130,
"parent_rev": "516e261bd7808ede9c98eaaec8d27d04e2d01195", "commit_rev":
"fa8dc66fc643c757076297403cfa0afdae097b70"}

[commit-bot: I haz the power, 2016-11-30 15:52:28]

Description was changed from

==========
Change StyleFetchedImage from ResourceClient to ImageResourceObserver

This CL is a part of moving ResourceClient subclasses used for ImageResource
to ImageResourceObserver, in order to split ImageResource into Image-part and
Resource-part.

BUG=667641
==========

to

==========
Change StyleFetchedImage from ResourceClient to ImageResourceObserver

This CL is a part of moving ResourceClient subclasses used for ImageResource
to ImageResourceObserver, in order to split ImageResource into Image-part and
Resource-part.

BUG=667641
==========

[commit-bot: I haz the power, 2016-11-30 15:52:30]

Committed patchset #11 (id:200001)

[commit-bot: I haz the power, 2016-11-30 15:55:49]

Description was changed from

==========
Change StyleFetchedImage from ResourceClient to ImageResourceObserver

This CL is a part of moving ResourceClient subclasses used for ImageResource
to ImageResourceObserver, in order to split ImageResource into Image-part and
Resource-part.

BUG=667641
==========

to

==========
Change StyleFetchedImage from ResourceClient to ImageResourceObserver

This CL is a part of moving ResourceClient subclasses used for ImageResource
to ImageResourceObserver, in order to split ImageResource into Image-part and
Resource-part.

BUG=667641

Committed: https://crrev.com/37e0c8e928dfad9c455b5f2145a5e3e25e1350ca
Cr-Commit-Position: refs/heads/master@{#435269}
==========

[commit-bot: I haz the power, 2016-11-30 15:55:50]

Patchset 11 (id:??) landed as
https://crrev.com/37e0c8e928dfad9c455b5f2145a5e3e25e1350ca
Cr-Commit-Position: refs/heads/master@{#435269}