Fix an MQ parsing related crash

Fix an MQ parsing related crash

When MediaQueryParser used valuesList.clear(), clear() cleared the list's values, but not the offset of the current read value.
This fixes that.

BUG=353988
Possibly related to BUG=353982
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169630

[Yoav Weiss, 2014-03-19 17:30:18]

This fixes https://code.google.com/p/chromium/issues/detail?id=353982

[eseidel, 2014-03-19 17:45:12]

You didn't change CSSParserValues when you made your original change, why is
this part needed?

Also, please add a test.

[Yoav Weiss, 2014-03-19 17:51:04]

On 2014/03/19 17:45:12, eseidel wrote:
> You didn't change CSSParserValues when you made your original change, why is
> this part needed?

I've used the clear() method in my code. This method clears the stored values
but leaves the offset of the current value with its original value.
I can just fix my code (add a reset current value function), but I believe it is
better if clear() didn't leave the valueList in an unstable state.

> 
> Also, please add a test.
OK.

[eseidel, 2014-03-19 18:32:55]

lgtm

A test would still be good.

[Yoav Weiss, 2014-03-19 19:17:12]

On 2014/03/19 18:32:55, eseidel wrote:
> lgtm
> 
> A test would still be good.

Added a test.

[Yoav Weiss, 2014-03-19 19:19:01]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2014-03-19 19:19:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoav@yoav.ws/204333005/10001

[commit-bot: I haz the power, 2014-03-19 20:27:13]

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

[commit-bot: I haz the power, 2014-03-19 20:27:13]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_dbg

[Yoav Weiss, 2014-03-19 21:28:02]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2014-03-19 21:28:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoav@yoav.ws/204333005/10001

[commit-bot: I haz the power, 2014-03-19 21:46:09]

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

[commit-bot: I haz the power, 2014-03-19 21:46:10]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_dbg

[Yoav Weiss, 2014-03-19 21:50:12]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2014-03-19 21:50:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoav@yoav.ws/204333005/10001

[commit-bot: I haz the power, 2014-03-19 23:04:52]

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

[commit-bot: I haz the power, 2014-03-19 23:04:52]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_dbg

[Yoav Weiss, 2014-03-20 06:00:31]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2014-03-20 06:00:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoav@yoav.ws/204333005/10001

[commit-bot: I haz the power, 2014-03-20 07:00:36]

Change committed as 169630