Loading: split MockFontResourceClient from MockResourceClients

Loading: split MockFontResourceClient from MockResourceClients

To remove additional dependency from core/fetch/ to FontResource,
split MockFontResourceClient from MockResourceClients, and place
it under core/loader/resource/.

BUG=655920

[Takashi Toyoshima, 2016-11-17 07:24:31]

Description was changed from

==========
Loading: split MockFontResourceClient from MockResourceClients

To remove additional dependency from core/fetch/ to FontResource,
split MockFontResourceClient from MockResourceClients, and place
it under core/loader/resource/.

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

to

==========
Loading: split MockFontResourceClient from MockResourceClients

To remove additional dependency from core/fetch/ to FontResource,
split MockFontResourceClient from MockResourceClients, and place
it under core/loader/resource/.

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

[Takashi Toyoshima, 2016-11-17 07:24:31]

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

[Takashi Toyoshima, 2016-11-17 07:25:25]

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

[commit-bot: I haz the power, 2016-11-17 07:25:53]

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

[Takashi Toyoshima, 2016-11-17 07:26:25]

Since Shao's CL introduced one more dependency from core/fetch to FontResource,
let me submit this additional change before submitting FontResource moving CL.

[yhirano, 2016-11-17 07:30:48]

lgtm

https://codereview.chromium.org/2513553002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/resource/MockFontResourceClient.h
(right):

https://codereview.chromium.org/2513553002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/resource/MockFontResourceClient.h:8:
#include "core/fetch/FontResource.h"
#include "core/fetch/Resource.h"

https://codereview.chromium.org/2513553002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/resource/MockFontResourceClient.h:9:
#include "platform/heap/Handle.h"
#include "wtf/text/WTFString.h"

[hiroshige, 2016-11-17 09:05:56]

lgtm

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

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

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

Dry run: This issue passed the CQ dry run.

[Takashi Toyoshima, 2016-11-17 10:18:09]

https://codereview.chromium.org/2513553002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/resource/MockFontResourceClient.h
(right):

https://codereview.chromium.org/2513553002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/resource/MockFontResourceClient.h:8:
#include "core/fetch/FontResource.h"
On 2016/11/17 07:30:48, yhirano wrote:
> #include "core/fetch/Resource.h"

Done.

https://codereview.chromium.org/2513553002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/resource/MockFontResourceClient.h:9:
#include "platform/heap/Handle.h"
On 2016/11/17 07:30:47, yhirano wrote:
> #include "wtf/text/WTFString.h"

Done.

[Takashi Toyoshima, 2016-11-17 10:18:17]

The CQ bit was checked by toyoshim@chromium.org

[Takashi Toyoshima, 2016-11-17 10:18:17]

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/2513553002/#ps20001
(title: "review #6")

[commit-bot: I haz the power, 2016-11-17 10:18:44]

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-17 10:25:53]

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

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

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

[Takashi Toyoshima, 2016-11-17 10:26:34]

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

[Takashi Toyoshima, 2016-11-17 10:26:59]

+tkent for BUILD.gn again

[tkent, 2016-11-17 22:24:04]

lgtm

[Takashi Toyoshima, 2016-11-29 07:41:25]

Patchset #3 (id:40001) has been deleted

[Takashi Toyoshima, 2016-11-29 07:43:22]

Sorry for being lazy to handle this patch, but I'm finally back from P1 bug
shooting.
Since the CL that this patch tried to fix was reverted, I'd just close this.