Avoid invoking ResourceFetcher::cachedResource() with a null URL

Avoid invoking ResourceFetcher::cachedResource() with a null URL

Invalid CSS image URLs end up with a null m_absoluteURL internally.
ResourceFetcher::cachedResource() cannot deal with null URLs so check
before calling it.

BUG=622703
Committed: https://crrev.com/2b4803484f5e55299bf6f5ea1bd95639f88e75b1
Cr-Commit-Position: refs/heads/master@{#404111}

[alancutter (OOO until 2018), 2016-07-07 05:36:21]

alancutter@chromium.org changed reviewers:
+ timloh@chromium.org

[alancutter (OOO until 2018), 2016-07-07 05:36:40]

Description was changed from

==========
Null check m_absoluteURL before invoking ResourceFetcher::cachedResource()

Invalid CSS image URLs end up with a null m_absoluteURL internally.
ResourceFetcher::cachedResource() cannot deal with null URLs so check
before calling it.

BUG=622703
==========

to

==========
Avoid invoking ResourceFetcher::cachedResource() with a null URL

Invalid CSS image URLs end up with a null m_absoluteURL internally.
ResourceFetcher::cachedResource() cannot deal with null URLs so check
before calling it.

BUG=622703
==========

[Timothy Loh, 2016-07-07 05:41:11]

On 2016/07/07 05:36:22, alancutter wrote:

Test?

[alancutter (OOO until 2018), 2016-07-07 05:42:04]

On 2016/07/07 at 05:41:11, timloh wrote:
> On 2016/07/07 05:36:22, alancutter wrote:
> 
> Test?

Test.

[alancutter (OOO until 2018), 2016-07-07 05:43:06]

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

[commit-bot: I haz the power, 2016-07-07 05:43:17]

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

[Timothy Loh, 2016-07-07 05:44:47]

On 2016/07/07 05:42:04, alancutter wrote:
> On 2016/07/07 at 05:41:11, timloh wrote:
> > On 2016/07/07 05:36:22, alancutter wrote:
> > 
> > Test?
> 
> Test.

lgtm

[commit-bot: I haz the power, 2016-07-07 07:47:18]

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

[commit-bot: I haz the power, 2016-07-07 07:47:19]

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

[alancutter (OOO until 2018), 2016-07-07 08:23:40]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2016-07-07 08:23:55]

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-07-07 09:22:51]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-07 09:22:59]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-07 09:25:24]

Description was changed from

==========
Avoid invoking ResourceFetcher::cachedResource() with a null URL

Invalid CSS image URLs end up with a null m_absoluteURL internally.
ResourceFetcher::cachedResource() cannot deal with null URLs so check
before calling it.

BUG=622703
==========

to

==========
Avoid invoking ResourceFetcher::cachedResource() with a null URL

Invalid CSS image URLs end up with a null m_absoluteURL internally.
ResourceFetcher::cachedResource() cannot deal with null URLs so check
before calling it.

BUG=622703

Committed: https://crrev.com/2b4803484f5e55299bf6f5ea1bd95639f88e75b1
Cr-Commit-Position: refs/heads/master@{#404111}
==========

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

Patchset 3 (id:??) landed as
https://crrev.com/2b4803484f5e55299bf6f5ea1bd95639f88e75b1
Cr-Commit-Position: refs/heads/master@{#404111}