flex shorthand incorrectly accepting 1 auto 1 (i.e. flex-basis in the middle)

flex shorthand incorrectly accepting 1 auto 1 (i.e. flex-basis in the middle)

This patch is in continuation with the given CL
https://codereview.chromium.org/396263003/
which fixes the bug of flex shorthand incorrectly accepting
values in which flex-basis is in the middle and does not have value
"auto". It does not took care of the case 1 auto 1 which according to
the grammar of te flex shorthand is incorrect.

This patch is adding support for the given missed case "1 auto 1"
by modifying the condition to check for if flex-basis is in the
middle or not in the argument list for all the different types of
values of flex-basis.

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

[harpreet.sk, 2014-07-19 13:33:24]

Sorry i missed this case in previous patch.... ptal...

[tony, 2014-07-21 16:17:56]

Please add a test case to css3/flexbox/flex-property-parsing.html.

[harpreet.sk, 2014-07-22 11:25:18]

On 2014/07/21 16:17:56, tony wrote:
> Please add a test case to css3/flexbox/flex-property-parsing.html.

Test case added in css3/flexbox/flex-property-parsing.html... ptal..

[tony, 2014-07-22 17:21:01]

LGTM

[harpreet.sk, 2014-07-22 18:06:56]

@leviw - require owner's lgtm

[leviw_travelin_and_unemployed, 2014-07-22 18:19:23]

rslgtm

[harpreet.sk, 2014-07-22 18:37:23]

The CQ bit was checked by harpreet.sk@samsung.com

[commit-bot: I haz the power, 2014-07-22 18:38:26]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/harpreet.sk@samsung.com/408683002/20001

[commit-bot: I haz the power, 2014-07-22 20:22:51]

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

[commit-bot: I haz the power, 2014-07-22 21:14:24]

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

[commit-bot: I haz the power, 2014-07-22 21:14:25]

Try jobs failed on following builders:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[harpreet.sk, 2014-07-23 06:43:16]

The CQ bit was checked by harpreet.sk@samsung.com

[commit-bot: I haz the power, 2014-07-23 06:43:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/harpreet.sk@samsung.com/408683002/20001

[commit-bot: I haz the power, 2014-07-23 06:44:41]

Change committed as 178738