Ignore:
Timestamp:
03/22/12 20:07:24 (3 years ago)
Author:
commit-queue@webkit.org
Message:

Fieldset unexpectedly stretches to minimum intrinsic width
https://bugs.webkit.org/show_bug.cgi?id=79128

Patch by SravanKumar Sandela <ssandela@innominds.com> on 2012-03-22
Reviewed by Julien Chaffraix.

Source/WebCore:

Fieldset element width will now check if css width is specified explicitly
before stretching to minimum intrinsic width. The reference
can be taken from IE9, instead of FF(FF acknowledged the broken behavior).

Tests: fast/forms/fieldset-width-nostretch-ifspecified-expected.html

fast/forms/fieldset-width-nostretch-ifspecified.html

  • rendering/RenderFieldset.cpp:

(WebCore::RenderFieldset::stretchesToMinIntrinsicLogicalWidth):
(WebCore):

  • rendering/RenderFieldset.h:

(RenderFieldset):

LayoutTests:

Ref tests to check no stretching to intrinsic width if width is explicitly specified.
Currently, calc width is Work-In-Progress, so it wont be supported now, and these ref
tests should automatically take care once Webkit enables it. Need to keep an eye on it.

  • fast/forms/fieldset-width-nostretch-ifspecified-expected.html: Added.
  • fast/forms/fieldset-width-nostretch-ifspecified.html: Added.
File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r111808 r111818  
     12012-03-22  SravanKumar Sandela  <ssandela@innominds.com> 
     2 
     3        Fieldset unexpectedly stretches to minimum intrinsic width 
     4        https://bugs.webkit.org/show_bug.cgi?id=79128 
     5 
     6        Reviewed by Julien Chaffraix. 
     7 
     8        Ref tests to check no stretching to intrinsic width if width is explicitly specified. 
     9        Currently, calc width is Work-In-Progress, so it wont be supported now, and these ref 
     10        tests should automatically take care once Webkit enables it. Need to keep an eye on it.  
     11 
     12        * fast/forms/fieldset-width-nostretch-ifspecified-expected.html: Added. 
     13        * fast/forms/fieldset-width-nostretch-ifspecified.html: Added. 
     14 
    1152012-03-22  Tom Sepez  <tsepez@chromium.org> 
    216 
Note: See TracChangeset for help on using the changeset viewer.