Fix possible use of garbage value in acquirePaintingResource

Fix possible use of garbage value in acquirePaintingResource

If ASSERT_NOT_REACHED is compiled out (like in release builds), then
hasFallback can be used uninitialized below.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169000

[Mostyn Bramley-Moore, 2014-03-11 23:08:00]

PTAL...

[eae, 2014-03-11 23:13:58]

Seems reasonable. Is there a bug or test for this?

[Mostyn Bramley-Moore, 2014-03-11 23:16:42]

No bug that I know of- I found this while testing static analyzer tools.

[eae, 2014-03-11 23:18:54]

On 2014/03/11 23:16:42, Mostyn Bramley-Moore wrote:
> No bug that I know of- I found this while testing static analyzer tools.

Ok

LGTM

[Mostyn Bramley-Moore, 2014-03-11 23:19:56]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2014-03-11 23:20:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/196113002/1

[Timothy Loh, 2014-03-11 23:35:38]

On 2014/03/11 23:16:42, Mostyn Bramley-Moore wrote:
> No bug that I know of- I found this while testing static analyzer tools.

This seems fine but feels a little weird; can't we in general assume debug
assertions will hold? OK (lgtm) to silence a static analyzer though.

[commit-bot: I haz the power, 2014-03-12 03:13:25]

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

[commit-bot: I haz the power, 2014-03-12 03:13:25]

Try jobs failed on following builders: linux_blink_rel

[Mostyn Bramley-Moore, 2014-03-12 08:24:53]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2014-03-12 08:25:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/196113002/1

[commit-bot: I haz the power, 2014-03-12 09:37:56]

Change committed as 169000