Remove PlatformImpl::createURLLoader

Remove PlatformImpl::createURLLoader

PlatformImpl::createURLLoader is used only by tests. In that case,
resource_dispatcher is null and the returned url loader should be
used only for local resources such as data URLs. This CL moves
the implementation to TestBlinkWebUnitTestSupport to make it explicit.

BUG=603396
Committed: https://crrev.com/6e4dac6c4668c879b0034d8ab4ed82a96c7a51be
Cr-Commit-Position: refs/heads/master@{#389421}

[yhirano, 2016-04-25 05:30:16]

Description was changed from

==========
[DO NOT COMMIT] test if PlatformImpl::createURLLoader is used.

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

to

==========
PlatformImpl::createURLLoader is only for tests

PlatformImpl::createURLLoader is only for tests is only used by tests.
In that case, resource_dispatcher is null and the returned url loader
should be used only for local resources such as data URLs. This
CL make it explicit.

BUGS=603396


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

[yhirano, 2016-04-25 05:30:37]

Description was changed from

==========
PlatformImpl::createURLLoader is only for tests

PlatformImpl::createURLLoader is only for tests is only used by tests.
In that case, resource_dispatcher is null and the returned url loader
should be used only for local resources such as data URLs. This
CL make it explicit.

BUGS=603396


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

to

==========
PlatformImpl::createURLLoader is only for tests

PlatformImpl::createURLLoader is only for tests is only used by tests.
In that case, resource_dispatcher is null and the returned url loader
should be used only for local resources such as data URLs. This
CL make it explicit.

BUG=603396
==========

[yhirano, 2016-04-25 05:47:23]

Description was changed from

==========
PlatformImpl::createURLLoader is only for tests

PlatformImpl::createURLLoader is only for tests is only used by tests.
In that case, resource_dispatcher is null and the returned url loader
should be used only for local resources such as data URLs. This
CL make it explicit.

BUG=603396
==========

to

==========
Remove PlatformImpl::createURLLoader

PlatformImpl::createURLLoader is only for tests is only used by tests.
In that case, resource_dispatcher is null and the returned url loader
should be used only for local resources such as data URLs. This
CL moves the implementation to TestBlinkWebUnitTestSupport to make
it explicit.

BUG=603396
==========

[yhirano, 2016-04-25 05:48:04]

Description was changed from

==========
Remove PlatformImpl::createURLLoader

PlatformImpl::createURLLoader is only for tests is only used by tests.
In that case, resource_dispatcher is null and the returned url loader
should be used only for local resources such as data URLs. This
CL moves the implementation to TestBlinkWebUnitTestSupport to make
it explicit.

BUG=603396
==========

to

==========
Remove PlatformImpl::createURLLoader

PlatformImpl::createURLLoader is used only by tests. In that case,
resource_dispatcher is null and the returned url loader should be
used only for local resources such as data URLs. This CL moves
the implementation to TestBlinkWebUnitTestSupport to make
it explicit.

BUG=603396
==========

[yhirano, 2016-04-25 05:48:18]

Description was changed from

==========
Remove PlatformImpl::createURLLoader

PlatformImpl::createURLLoader is used only by tests. In that case,
resource_dispatcher is null and the returned url loader should be
used only for local resources such as data URLs. This CL moves
the implementation to TestBlinkWebUnitTestSupport to make
it explicit.

BUG=603396
==========

to

==========
Remove PlatformImpl::createURLLoader

PlatformImpl::createURLLoader is used only by tests. In that case,
resource_dispatcher is null and the returned url loader should be
used only for local resources such as data URLs. This CL moves
the implementation to TestBlinkWebUnitTestSupport to make it explicit.

BUG=603396
==========

[yhirano, 2016-04-25 06:12:47]

Patchset #5 (id:80001) has been deleted

[yhirano, 2016-04-25 06:50:32]

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

[kinuko, 2016-04-25 06:56:06]

\o/ yes, lgtm

[yhirano, 2016-04-25 07:04:12]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2016-04-25 07:04:27]

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

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

Description was changed from

==========
Remove PlatformImpl::createURLLoader

PlatformImpl::createURLLoader is used only by tests. In that case,
resource_dispatcher is null and the returned url loader should be
used only for local resources such as data URLs. This CL moves
the implementation to TestBlinkWebUnitTestSupport to make it explicit.

BUG=603396
==========

to

==========
Remove PlatformImpl::createURLLoader

PlatformImpl::createURLLoader is used only by tests. In that case,
resource_dispatcher is null and the returned url loader should be
used only for local resources such as data URLs. This CL moves
the implementation to TestBlinkWebUnitTestSupport to make it explicit.

BUG=603396
==========

[commit-bot: I haz the power, 2016-04-25 07:08:05]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-04-25 07:10:02]

Description was changed from

==========
Remove PlatformImpl::createURLLoader

PlatformImpl::createURLLoader is used only by tests. In that case,
resource_dispatcher is null and the returned url loader should be
used only for local resources such as data URLs. This CL moves
the implementation to TestBlinkWebUnitTestSupport to make it explicit.

BUG=603396
==========

to

==========
Remove PlatformImpl::createURLLoader

PlatformImpl::createURLLoader is used only by tests. In that case,
resource_dispatcher is null and the returned url loader should be
used only for local resources such as data URLs. This CL moves
the implementation to TestBlinkWebUnitTestSupport to make it explicit.

BUG=603396

Committed: https://crrev.com/6e4dac6c4668c879b0034d8ab4ed82a96c7a51be
Cr-Commit-Position: refs/heads/master@{#389421}
==========

[commit-bot: I haz the power, 2016-04-25 07:10:03]

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