Loading: move V8DOMActivityLogger dependency from core/fetch

Loading: move V8DOMActivityLogger dependency from core/fetch

Now, core/fetch has a dependency to bindings/core/, but this won't be
permitted once the directory is moved to platform/loader/fetch.

This patch moves this last unexpected dependency, and make it
finally possible to move under platform/.

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

[Takashi Toyoshima, 2016-12-15 11:30:10]

Description was changed from

==========
Loading: move V8DOMActivityLogger dependency from core/fetch to core/loader

Now, core/fetch has a dependency to bindings/core/, but this won't be
permitted once the directory is moved to platform/loader/fetch.

This patch moves this last unexpected dependency, and make it
finally possible to move under platform/.

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

to

==========
Loading: move V8DOMActivityLogger dependency from core/fetch to core/loader

Now, core/fetch has a dependency to bindings/core/, but this won't be
permitted once the directory is moved to platform/loader/fetch.

This patch moves this last unexpected dependency, and make it
finally possible to move under platform/.

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

[Takashi Toyoshima, 2016-12-15 11:30:10]

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

[Takashi Toyoshima, 2016-12-15 11:30:20]

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

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

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-12-15 11:31:01]

Description was changed from

==========
Loading: move V8DOMActivityLogger dependency from core/fetch to core/loader

Now, core/fetch has a dependency to bindings/core/, but this won't be
permitted once the directory is moved to platform/loader/fetch.

This patch moves this last unexpected dependency, and make it
finally possible to move under platform/.

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

to

==========
Loading: move V8DOMActivityLogger dependency from core/fetch

Now, core/fetch has a dependency to bindings/core/, but this won't be
permitted once the directory is moved to platform/loader/fetch.

This patch moves this last unexpected dependency, and make it
finally possible to move under platform/.

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

[commit-bot: I haz the power, 2016-12-15 12:56:37]

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

[commit-bot: I haz the power, 2016-12-15 12:56:37]

Dry run: This issue passed the CQ dry run.

[Takashi Toyoshima, 2016-12-19 05:48:12]

ptal

[yhirano, 2016-12-19 11:44:47]

https://codereview.chromium.org/2578983002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/fetch/FetchContext.h (right):

https://codereview.chromium.org/2578983002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FetchContext.h:44: namespace wtf {
wtf/Forward.h

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

https://codereview.chromium.org/2578983002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/FrameFetchContext.h:42: namespace wtf {
ditto

https://codereview.chromium.org/2578983002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/fetch/FetchContext.h (right):

https://codereview.chromium.org/2578983002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/fetch/FetchContext.h:135: bool forPreload);
Please pass an enum instead of boolean.

[Takashi Toyoshima, 2016-12-19 12:58:54]

https://codereview.chromium.org/2578983002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/fetch/FetchContext.h (right):

https://codereview.chromium.org/2578983002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FetchContext.h:44: namespace wtf {
On 2016/12/19 11:44:46, yhirano wrote:
> wtf/Forward.h

Done.

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

https://codereview.chromium.org/2578983002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/FrameFetchContext.h:42: namespace wtf {
On 2016/12/19 11:44:46, yhirano wrote:
> ditto

Done.

https://codereview.chromium.org/2578983002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/fetch/FetchContext.h (right):

https://codereview.chromium.org/2578983002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/fetch/FetchContext.h:135: bool forPreload);
I feel the enum is better to be defined in FetchRequest.
So, I want to leave TODOs in this CL, and fix it in a separate CL.

[yhirano, 2016-12-20 12:14:00]

lgtm

It might be good to add an entry in core/fetch/DEPS to forbid dependency to
bindings/ but it's optional.

[Takashi Toyoshima, 2016-12-20 13:54:08]

The CQ bit was checked by toyoshim@chromium.org

[Takashi Toyoshima, 2016-12-20 13:54:08]

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/2578983002/#ps100001
(title: "DEPS update")

[commit-bot: I haz the power, 2016-12-20 13:54:24]

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-20 15:18:30]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1482242048371620,
"parent_rev": "43973d1fb56a83de27e054e26f01b3b9154628a9", "commit_rev":
"8b83dc1f81d7672f2a2e75ccc3d94699c34b4a8a"}

[commit-bot: I haz the power, 2016-12-20 15:18:56]

Description was changed from

==========
Loading: move V8DOMActivityLogger dependency from core/fetch

Now, core/fetch has a dependency to bindings/core/, but this won't be
permitted once the directory is moved to platform/loader/fetch.

This patch moves this last unexpected dependency, and make it
finally possible to move under platform/.

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

to

==========
Loading: move V8DOMActivityLogger dependency from core/fetch

Now, core/fetch has a dependency to bindings/core/, but this won't be
permitted once the directory is moved to platform/loader/fetch.

This patch moves this last unexpected dependency, and make it
finally possible to move under platform/.

BUG=655920

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

[commit-bot: I haz the power, 2016-12-20 15:18:57]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-12-20 15:20:37]

Description was changed from

==========
Loading: move V8DOMActivityLogger dependency from core/fetch

Now, core/fetch has a dependency to bindings/core/, but this won't be
permitted once the directory is moved to platform/loader/fetch.

This patch moves this last unexpected dependency, and make it
finally possible to move under platform/.

BUG=655920

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

to

==========
Loading: move V8DOMActivityLogger dependency from core/fetch

Now, core/fetch has a dependency to bindings/core/, but this won't be
permitted once the directory is moved to platform/loader/fetch.

This patch moves this last unexpected dependency, and make it
finally possible to move under platform/.

BUG=655920

Committed: https://crrev.com/5daea93d2ae560a310f341015ec9d21b0fcfea77
Cr-Commit-Position: refs/heads/master@{#439806}
==========

[commit-bot: I haz the power, 2016-12-20 15:20:38]

Patchset 6 (id:??) landed as
https://crrev.com/5daea93d2ae560a310f341015ec9d21b0fcfea77
Cr-Commit-Position: refs/heads/master@{#439806}