Only count legacy parser usage if legacy parser had effect.

Only count legacy parser usage if legacy parser had effect.

We would otherwise also count if its just trimming whitespaces.

R=adamk@chromium.org
BUG=chromium:618595
Committed: https://crrev.com/8b67a002231c222314170a13187e31a2b361ef91
Cr-Commit-Position: refs/heads/master@{#37197}

[adamk, 2016-06-21 16:11:40]

Change seems fine, but how about adding a few tests? You could do this pretty
easily from a cctest.

[Yang, 2016-06-22 12:56:46]

On 2016/06/21 16:11:40, adamk wrote:
> Change seems fine, but how about adding a few tests? You could do this pretty
> easily from a cctest.

Added test as suggested

[adamk, 2016-06-22 17:41:28]

The CQ bit was checked by adamk@chromium.org

[adamk, 2016-06-22 17:41:28]

lgtm

[commit-bot: I haz the power, 2016-06-22 17:41:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2080183003/20001

[commit-bot: I haz the power, 2016-06-22 18:06:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-22 18:08:00]

Description was changed from

==========
Only count legacy parser usage if legacy parser had effect.

We would otherwise also count if its just trimming whitespaces.

R=adamk@chromium.org
BUG=chromium:618595
==========

to

==========
Only count legacy parser usage if legacy parser had effect.

We would otherwise also count if its just trimming whitespaces.

R=adamk@chromium.org
BUG=chromium:618595

Committed: https://crrev.com/8b67a002231c222314170a13187e31a2b361ef91
Cr-Commit-Position: refs/heads/master@{#37197}
==========

[commit-bot: I haz the power, 2016-06-22 18:08:01]

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