Refactor <paint> URL resolution in SVGResources

Refactor <paint> URL resolution in SVGResources

Hoist the has-url check out of paintingResourceFromSVGPaint(), and
remove |hasPendingResource| - consider all null-returns as having pending
resources. (This means "incorrect" <paint> URL references are considered
pending, just as URL references for other resource types.)

BUG=454767
Review-Url: https://codereview.chromium.org/2746933002
Cr-Commit-Position: refs/heads/master@{#456374}
Committed: https://chromium.googlesource.com/chromium/src/+/ec5133da581afeb7e71904f528df9ae4966ccfaf

[fs, 2017-03-13 12:24:35]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-13 12:24:43]

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

[fs, 2017-03-13 12:27:41]

Description was changed from

==========
Refactor <paint> URL resolution in SVGResources

Hoist the has-url check out of paintingResourceFromSVGPaint(), and
remove the |hasPendingResource| - consider all null-returns as having
pending resources. (This means "incorrect" <paint> URL references are
considered pending, just as URL references for other resource types.)

BUG=454767
==========

to

==========
Refactor <paint> URL resolution in SVGResources

Hoist the has-url check out of paintingResourceFromSVGPaint(), and
remove |hasPendingResource| - consider all null-returns as having pending
resources. (This means "incorrect" <paint> URL references are considered
pending, just as URL references for other resource types.)

BUG=454767
==========

[fs, 2017-03-13 13:25:10]

fs@opera.com changed reviewers:
+ pdr@chromium.org, schenney@chromium.org

[commit-bot: I haz the power, 2017-03-13 13:44:52]

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

[commit-bot: I haz the power, 2017-03-13 13:44:53]

Dry run: This issue passed the CQ dry run.

[Stephen Chennney, 2017-03-13 14:24:10]

The CQ bit was checked by schenney@chromium.org

[Stephen Chennney, 2017-03-13 14:24:11]

lgtm

[commit-bot: I haz the power, 2017-03-13 14:24:31]

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, 2017-03-13 14:28:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489415050764560, "parent_rev":
"8a4451c911ee05e56000474c58bfad89f744a35e", "commit_rev":
"ec5133da581afeb7e71904f528df9ae4966ccfaf"}

[commit-bot: I haz the power, 2017-03-13 14:29:03]

Description was changed from

==========
Refactor <paint> URL resolution in SVGResources

Hoist the has-url check out of paintingResourceFromSVGPaint(), and
remove |hasPendingResource| - consider all null-returns as having pending
resources. (This means "incorrect" <paint> URL references are considered
pending, just as URL references for other resource types.)

BUG=454767
==========

to

==========
Refactor <paint> URL resolution in SVGResources

Hoist the has-url check out of paintingResourceFromSVGPaint(), and
remove |hasPendingResource| - consider all null-returns as having pending
resources. (This means "incorrect" <paint> URL references are considered
pending, just as URL references for other resource types.)

BUG=454767

Review-Url: https://codereview.chromium.org/2746933002
Cr-Commit-Position: refs/heads/master@{#456374}
Committed:
https://chromium.googlesource.com/chromium/src/+/ec5133da581afeb7e71904f528df...
==========

[commit-bot: I haz the power, 2017-03-13 14:29:04]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ec5133da581afeb7e71904f528df...