Ignore ::first-letter from ancestors in grids and flexboxes

Ignore ::first-letter from ancestors in grids and flexboxes

According to the grid and flexbox specs:
"the ::first-line and ::first-letter pseudo-elements do not apply to
grid/flex containers".
http://dev.w3.org/csswg/css-grid/#grid-containers
http://dev.w3.org/csswg/css-flexbox/#flex-containers

This was almost working right, except in the case that an ancestor was
setting the ::first-letter pseudo-element.

Modified RenderBlock::updateFirstLetter() in order to stop looking for
the first text child when you reach a grid or flexbox.
Added a few more cases to the current tests in order to check this
behavior.

TEST=css3/flexbox/flexbox-ignore-container-firstLetter.html
TEST=fast/css-grid-layout/grid-container-ignore-first-letter.html

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

[Manuel Rego, 2014-11-04 21:45:57]

rego@igalia.com changed reviewers:
+ cbiesinger@chromium.org, jchaffraix@chromium.org, leviw@chromium.org

[leviw_travelin_and_unemployed, 2014-11-05 06:55:35]

leviw@chromium.org changed reviewers:
+ dsinclair@chromium.org

[leviw_travelin_and_unemployed, 2014-11-05 06:55:36]

Please add a link to the text in the spec you quoted in your description. LGTM.

https://codereview.chromium.org/686173006/diff/20001/LayoutTests/css3/flexbox...
File LayoutTests/css3/flexbox/flexbox-ignore-container-firstLetter.html (right):

https://codereview.chromium.org/686173006/diff/20001/LayoutTests/css3/flexbox...
LayoutTests/css3/flexbox/flexbox-ignore-container-firstLetter.html:6:
.flexboxFirsLetter::first-letter { line-height: 200px; }
Typo: "flexboxFirsLetter"

[Manuel Rego, 2014-11-05 08:13:01]

Thanks for the review. I've just added the links in the description.

https://codereview.chromium.org/686173006/diff/20001/LayoutTests/css3/flexbox...
File LayoutTests/css3/flexbox/flexbox-ignore-container-firstLetter.html (right):

https://codereview.chromium.org/686173006/diff/20001/LayoutTests/css3/flexbox...
LayoutTests/css3/flexbox/flexbox-ignore-container-firstLetter.html:6:
.flexboxFirsLetter::first-letter { line-height: 200px; }
On 2014/11/05 06:55:36, leviw wrote:
> Typo: "flexboxFirsLetter"

Acknowledged.

[Manuel Rego, 2014-11-05 08:13:11]

The CQ bit was checked by rego@igalia.com

[commit-bot: I haz the power, 2014-11-05 08:13:44]

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

[commit-bot: I haz the power, 2014-11-05 09:16:51]

Committed patchset #3 (id:40001) as 184858