Marked parsing as failed for invalid compound.

Marked parsing as failed for invalid compound.

If we return nullptr trying to consume a compound, and all tokens have
been consumed, we don't recognize that as a parse failure if a valid
selector precedes the compound. Set m_failedParsing to true to notice
we have an invalid selector.

R=timloh@chromium.org
BUG=581257
Committed: https://crrev.com/acce4131402e62abdee469c445849c1036357e6e
Cr-Commit-Position: refs/heads/master@{#371760}

[rune, 2016-01-27 00:18:14]

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

[commit-bot: I haz the power, 2016-01-27 00:19:09]

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

[commit-bot: I haz the power, 2016-01-27 01:40:29]

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

[commit-bot: I haz the power, 2016-01-27 01:40:29]

Dry run: This issue passed the CQ dry run.

[Timothy Loh, 2016-01-27 08:07:46]

lgtm (btw you didn't actually send this out)

[rune, 2016-01-27 08:28:38]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2016-01-27 08:29:12]

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

[commit-bot: I haz the power, 2016-01-27 08:35:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-27 08:36:52]

Description was changed from

==========
Marked parsing as failed for invalid compound.

If we return nullptr trying to consume a compound, and all tokens have
been consumed, we don't recognize that as a parse failure if a valid
selector precedes the compound. Set m_failedParsing to true to notice
we have an invalid selector.

R=timloh@chromium.org
BUG=581257
==========

to

==========
Marked parsing as failed for invalid compound.

If we return nullptr trying to consume a compound, and all tokens have
been consumed, we don't recognize that as a parse failure if a valid
selector precedes the compound. Set m_failedParsing to true to notice
we have an invalid selector.

R=timloh@chromium.org
BUG=581257

Committed: https://crrev.com/acce4131402e62abdee469c445849c1036357e6e
Cr-Commit-Position: refs/heads/master@{#371760}
==========

[commit-bot: I haz the power, 2016-01-27 08:36:53]

Patchset 1 (id:??) landed as
https://crrev.com/acce4131402e62abdee469c445849c1036357e6e
Cr-Commit-Position: refs/heads/master@{#371760}

[rune, 2016-01-27 08:53:13]

On 2016/01/27 08:07:46, Timothy Loh wrote:
> lgtm (btw you didn't actually send this out)

Sorry, it did not appear in my unsent category.