[SVG 1/4] Move Resource::finish() for data URLs from requestResource() to Resource::load()

Move Resource::finish() for data URLs from requestResource() to Resource::load()

Previously, for both SubstituteData and data URLs,
1. Resource and ResourceResponse are created and
2. Resource::finish() is always called
in ResourceFetcher::resourceForStaticData() called from requestResource().

However, FontResource and ImageResource don't always start loading in
requestResource(), and thus Resource::finish() shouldn't be called in such
cases.

This CL
1. Creates Resource for a data URL by createResourceForLoading() called in
   the main path of requestResource(), and
2. Creates ResourceResponse and calls Resource::finish() in Resource::load().

To do this, most of ResourceFetcher::resourceForStaticData() is moved into
Resource::loadLocallyStaticData(), loadLocallyDataURL() and
loadLocallySubstituteData().

Resource for SubstituteData is still created in a separated path in
requestResource() and Resource::finish() is still called inside
requestResource(), because this is not a problem for SubstituteData.

This CL is [1/4] to load FontResource of data: URLs synchronously so that
all subresource loading of SVG image completes before loading of the SVG file.

BUG=382170

[hiroshige, 2016-03-02 22:36:14]

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

[hiroshige, 2016-03-02 22:36:24]

Description was changed from

==========
Call Resource::finish() for data: URLs in Resource::load()

BUG=382170
==========

to

==========
Call Resource::finish() for data: URLs in Resource::load()

BUG=382170
==========

[commit-bot: I haz the power, 2016-03-02 22:36:59]

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

[hiroshige, 2016-03-02 23:44:16]

Description was changed from

==========
Call Resource::finish() for data: URLs in Resource::load()

BUG=382170
==========

to

==========
Move Resource::finish() for data URLs from requestResource() to Resource::load()

Previously, for both SubstituteData and data URLs,
1. Resource and ResourceResponse are created and
2. Resource::finish() is always called
in ResourceFetcher::resourceForStaticData() called from requestResource().

However, FontResource and ImageResource don't always start loading in
requestResource(), and thus Resource::finish() shouldn't be called in such
cases.

This CL
1. Creates Resource for a data URL by createResourceForLoading() called in
   the main path of requestResource(), and
2. Creates ResourceResponse and calls Resource::finish() in Resource::load().

To do this, most of ResourceFetcher::resourceForStaticData() is moved into
Resource::loadLocallyStaticData(), loadLocallyDataURL() and
loadLocallySubstituteData().

Resource for SubstituteData is still created in a separated path in
requestResource() and Resource::finish() is still called inside
requestResource(), because this is not a problem for SubstituteData.

This CL is [1/4] to load FontResource of data: URLs synchronously so that
all subresource loading of SVG image completes before loading of the SVG file.

BUG=382170
==========

[commit-bot: I haz the power, 2016-03-02 23:50:01]

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

[commit-bot: I haz the power, 2016-03-02 23:50:01]

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

[hiroshige, 2016-03-03 00:16:04]

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

[commit-bot: I haz the power, 2016-03-03 00:16:42]

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