Clean up SizesAttributeParserTest and add some tests

Clean up SizesAttributeParserTest and add some tests

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

[Yoav Weiss, 2014-05-28 06:12:47]

This removed a stale comment and a FIXME, and adds a bunch of test cases that
were missing.

[eseidel, 2014-05-28 22:44:22]

lgtm

[commit-bot: I haz the power, 2014-05-28 22:44:53]

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

[commit-bot: I haz the power, 2014-05-28 23:03:52]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/9869)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/9321)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/9552)

[commit-bot: I haz the power, 2014-05-28 23:14:27]

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

[commit-bot: I haz the power, 2014-05-28 23:14:27]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/9323)

[Yoav Weiss, 2014-05-30 07:42:28]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2014-05-30 07:43:47]

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

[Yoav Weiss, 2014-05-30 08:08:32]

The CQ bit was unchecked by yoav@yoav.ws

[Yoav Weiss, 2014-05-30 08:53:03]

On 2014/05/30 08:08:32, Yoav Weiss wrote:
> The CQ bit was unchecked by mailto:yoav@yoav.ws

The tests fail because of a bug in the handling of zero length source size.
I've fixed it at https://codereview.chromium.org/302123002/
I'll land this CL after the fix has landed.

[Yoav Weiss, 2014-06-02 17:38:00]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2014-06-02 17:38:31]

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

[commit-bot: I haz the power, 2014-06-02 18:59:53]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/10202)

[Yoav Weiss, 2014-06-02 22:02:20]

The CQ bit was unchecked by yoav@yoav.ws

[Yoav Weiss, 2014-06-02 22:02:31]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2014-06-02 22:03:04]

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

[commit-bot: I haz the power, 2014-06-03 01:49:47]

Change committed as 175328