Set <dialog> width and height to 'fit-content'.

Set <dialog> width and height to 'fit-content'.

As per the spec: http://html5.org/r/8151

BUG=140205
TBR=tkent

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

[falken, 2013-08-23 06:59:33]

Kent-san, could you review please?

[tkent, 2013-08-25 21:41:24]

The change looks ok.  However fast/regions/dialog-autoheight.html is failing.

[falken, 2013-08-26 02:35:03]

Uploaded the patch to make regions/dialog-autoheight.html test for the case
where dialog has height: auto, as it was previously by default.

badea: can you check that the above change is correct?

[falken, 2013-08-26 03:13:40]

+Cătălin Badea

[Cătălin Badea, 2013-08-26 08:49:07]

On 2013/08/26 02:35:03, falken wrote:
> Uploaded the patch to make regions/dialog-autoheight.html test for the case
> where dialog has height: auto, as it was previously by default.
> 
> badea: can you check that the above change is correct?

Yes, it's ok. lgtm

I shall open a bug for CSS-Regions and 'height: fit-content' because the spec
doesn't
cover this case at the moment.

[falken, 2013-08-26 09:07:06]

badea: thanks!
tkent: TBR since I think you left for the day but were OK with the change as per
comment #2

[commit-bot: I haz the power, 2013-08-26 09:07:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/falken@chromium.org/23060031/1011

[tkent, 2013-08-26 09:22:31]

lgtm

[commit-bot: I haz the power, 2013-08-26 14:55:41]

Change committed as 156704