Remove all uses of display:block and display:inline-block.

Remove all uses of display:block and display:inline-block.

-Make display:flex, flex-direction: column, flex-shrink: 1 the default.
-Simplify StyleAdjuster::adjustStyleForAlignment to remove special cases we
won't need as we make flex the default and remove absolute positioning.
-Fix a bug this exposed in column flexboxes where we'd apply the wrong edge
of border/padding/margin.
-For now leave the default of align-items:stretch. The main change here is
that iframe/img will do width:auto the same as blocks (i.e. the width of
the parent). I think this is a good change, but we'll have to see how it feels
in practice.

R=eseidel@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/efa996dbae1fb7162316841f5a9f3eef58814e78

[eseidel, 2015-04-06 23:29:18]

eseidel@chromium.org changed reviewers:
+ eseidel@chromium.org

[eseidel, 2015-04-06 23:29:19]

lgtm

https://codereview.chromium.org/1061163002/diff/20001/sky/engine/core/css/res...
File sky/engine/core/css/resolver/StyleAdjuster.cpp (right):

https://codereview.chromium.org/1061163002/diff/20001/sky/engine/core/css/res...
sky/engine/core/css/resolver/StyleAdjuster.cpp:193:
style.setJustifySelf(parentStyle.justifyItems());
Should this be squashed into a single if now? These blocks look identical?

https://codereview.chromium.org/1061163002/diff/20001/sky/engine/core/css/res...
sky/engine/core/css/resolver/StyleAdjuster.cpp:199:
style.setAlignSelfOverflowAlignment(parentStyle.alignItemsOverflowAlignment());
This one too?  are these ifs even needed?

[ojan, 2015-04-06 23:41:50]

https://codereview.chromium.org/1061163002/diff/20001/sky/engine/core/css/res...
File sky/engine/core/css/resolver/StyleAdjuster.cpp (right):

https://codereview.chromium.org/1061163002/diff/20001/sky/engine/core/css/res...
sky/engine/core/css/resolver/StyleAdjuster.cpp:193:
style.setJustifySelf(parentStyle.justifyItems());
On 2015/04/06 23:29:19, eseidel wrote:
> Should this be squashed into a single if now? These blocks look identical?

They're reading/writing different values. justifyItems, justifySelf and
alignSelf. It's not clear to me we want to keep all or any of these properties,
but that will naturally fallout as we write the dart implementation of flexbox.
All this code will go away then.

[ojan, 2015-04-06 23:44:16]

Committed patchset #2 (id:20001) manually as
efa996dbae1fb7162316841f5a9f3eef58814e78 (presubmit successful).