WebFonts: merge fontLoaded() to notifyFinished()

WebFonts: merge fontLoaded() to notifyFinished()

Remove a FontResourceClient specific callback fontLoaded()
and use ResourceClient::notifyFinished() instead.

BUG=n/a

Committed: https://crrev.com/2d8b210dffbc41e44addc6a6c16a507d5de171a0
Cr-Commit-Position: refs/heads/master@{#388165}

[Takashi Toyoshima, 2016-04-18 07:46:52]

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

[commit-bot: I haz the power, 2016-04-18 07:47:17]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1894953002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1894953002/1

[Takashi Toyoshima, 2016-04-18 08:05:06]

Description was changed from

==========
WebFonts: merge fontLoaded() to notifyFinished()

Remove a FontResourceClient specific callback fontLoaded()
and use ResourceClient::notifyFinished() instead.

BUG=n/a
==========

to

==========
WebFonts: merge fontLoaded() to notifyFinished()

Remove a FontResourceClient specific callback fontLoaded()
and use ResourceClient::notifyFinished() instead.

BUG=n/a
==========

[Takashi Toyoshima, 2016-04-18 08:05:06]

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

[Takashi Toyoshima, 2016-04-18 08:07:09]

let me split this trivial changes from
https://codereview.chromium.org/1893233002/ because I need another investigation
to find a right root cause for the original bug.

[Kunihiko Sakamoto, 2016-04-18 08:18:59]

https://codereview.chromium.org/1894953002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/fetch/FontResource.cpp (right):

https://codereview.chromium.org/1894953002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FontResource.cpp:101:
c->notifyFinished(this);
This is called by above Resource::didAddClient(), isn't it?

[commit-bot: I haz the power, 2016-04-18 09:08:50]

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

[commit-bot: I haz the power, 2016-04-18 09:08:51]

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

[Takashi Toyoshima, 2016-04-18 12:42:03]

https://codereview.chromium.org/1894953002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/fetch/FontResource.cpp (right):

https://codereview.chromium.org/1894953002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FontResource.cpp:101:
c->notifyFinished(this);
On 2016/04/18 08:18:59, Kunihiko Sakamoto wrote:
> This is called by above Resource::didAddClient(), isn't it?

Done.

[Takashi Toyoshima, 2016-04-18 12:42:29]

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

[commit-bot: I haz the power, 2016-04-18 12:42:35]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1894953002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1894953002/20001

[commit-bot: I haz the power, 2016-04-18 13:47:27]

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

[commit-bot: I haz the power, 2016-04-18 13:47:27]

Dry run: This issue passed the CQ dry run.

[Kunihiko Sakamoto, 2016-04-19 01:11:17]

lgtm

[Takashi Toyoshima, 2016-04-19 05:05:05]

The CQ bit was checked by toyoshim@chromium.org

[commit-bot: I haz the power, 2016-04-19 05:05:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1894953002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1894953002/20001

[commit-bot: I haz the power, 2016-04-19 05:12:33]

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

[commit-bot: I haz the power, 2016-04-19 05:12:34]

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

[Takashi Toyoshima, 2016-04-19 06:30:12]

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

[Takashi Toyoshima, 2016-04-19 06:30:12]

kinuko@chromium.org: this change was reviewed by ksakamoto@, but could you take
another look for submitting?

[kinuko, 2016-04-19 07:51:14]

lgtm

[Takashi Toyoshima, 2016-04-19 08:05:48]

The CQ bit was checked by toyoshim@chromium.org

[commit-bot: I haz the power, 2016-04-19 08:05:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1894953002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1894953002/20001

[commit-bot: I haz the power, 2016-04-19 08:10:17]

Description was changed from

==========
WebFonts: merge fontLoaded() to notifyFinished()

Remove a FontResourceClient specific callback fontLoaded()
and use ResourceClient::notifyFinished() instead.

BUG=n/a
==========

to

==========
WebFonts: merge fontLoaded() to notifyFinished()

Remove a FontResourceClient specific callback fontLoaded()
and use ResourceClient::notifyFinished() instead.

BUG=n/a
==========

[commit-bot: I haz the power, 2016-04-19 08:10:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-19 08:11:07]

Description was changed from

==========
WebFonts: merge fontLoaded() to notifyFinished()

Remove a FontResourceClient specific callback fontLoaded()
and use ResourceClient::notifyFinished() instead.

BUG=n/a
==========

to

==========
WebFonts: merge fontLoaded() to notifyFinished()

Remove a FontResourceClient specific callback fontLoaded()
and use ResourceClient::notifyFinished() instead.

BUG=n/a

Committed: https://crrev.com/2d8b210dffbc41e44addc6a6c16a507d5de171a0
Cr-Commit-Position: refs/heads/master@{#388165}
==========

[commit-bot: I haz the power, 2016-04-19 08:11:08]

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