Bug 53743 - css style="max-width:none" works wrong
Summary: css style="max-width:none" works wrong
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: http://code.google.com/p/chromium/iss...
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-03 17:21 PST by Kenichi Ishibashi
Modified: 2011-02-14 05:14 PST (History)
5 users (show)

See Also:


Attachments
testcase (1.01 KB, text/html)
2011-02-03 17:22 PST, Kenichi Ishibashi
no flags Details
Patch (3.88 KB, patch)
2011-02-03 18:01 PST, Kenichi Ishibashi
bashi: commit-queue-
Details | Formatted Diff | Diff
Patch V1 (4.02 KB, patch)
2011-02-13 21:02 PST, Kenichi Ishibashi
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kenichi Ishibashi 2011-02-03 17:21:28 PST
Originally reported at: http://code.google.com/p/chromium/issues/detail?id=64511

'max-width: none' should have no effect because it is the initial state. For now, if style="max-width:none" is set to an element, it outer with is restricted to the with of the closest parent, which have width settings.

See the attached test case that the original reporter provided.
Comment 1 Kenichi Ishibashi 2011-02-03 17:22:15 PST
Created attachment 81153 [details]
testcase
Comment 2 Alexander Romanovich 2011-02-03 17:41:23 PST
See also bug 26809.
Comment 3 Kenichi Ishibashi 2011-02-03 18:01:20 PST
Created attachment 81162 [details]
Patch
Comment 4 Kenichi Ishibashi 2011-02-03 18:04:49 PST
Hi Alexander,

(In reply to comment #2)
> See also bug 26809.

Thank you for letting me the bug. I think the patch I just sent will fix the bug 26809, too.

Regards,
Comment 5 Alexander Romanovich 2011-02-03 20:25:33 PST
No problem. Thanks for providing a patch! Feel free to close the other bug if this solves it.
Comment 6 Kenichi Ishibashi 2011-02-09 00:08:55 PST
Hi hyatt,

Could you review this patch if you have time?

Thanks in advance,
Comment 7 Andreas Kling 2011-02-13 18:38:03 PST
Comment on attachment 81162 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=81162&action=review

Nice catch! r=me

For reference, max-height used to have the same problem, this was fixed in <http://trac.webkit.org/changeset/13462>

> Source/WebCore/ChangeLog:13
> +        (WebCore::CSSStyleSelector::applyProperty): Set length to be undefined.

This comment is more unspecific than helpful.
Comment 8 Kenichi Ishibashi 2011-02-13 21:02:49 PST
Created attachment 82284 [details]
Patch V1
Comment 9 Kenichi Ishibashi 2011-02-13 21:07:37 PST
Comment on attachment 81162 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=81162&action=review

Hi Andreas,

Thank you very much for review and your comments. I updated the patch.

Could someone cq+?

>> Source/WebCore/ChangeLog:13
>> +        (WebCore::CSSStyleSelector::applyProperty): Set length to be undefined.
> 
> This comment is more unspecific than helpful.

Revised.
Comment 10 Kenichi Ishibashi 2011-02-13 21:08:28 PST
Comment on attachment 81162 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=81162&action=review

Hi Andreas,

Thank you very much for review and your comments. I updated the patch.

Could someone cq+?

>>> Source/WebCore/ChangeLog:13
>>> +        (WebCore::CSSStyleSelector::applyProperty): Set length to be undefined.
>> 
>> This comment is more unspecific than helpful.
> 
> Revised.

Revised.
Comment 11 WebKit Commit Bot 2011-02-14 05:14:02 PST
Comment on attachment 82284 [details]
Patch V1

Clearing flags on attachment: 82284

Committed r78469: <http://trac.webkit.org/changeset/78469>
Comment 12 WebKit Commit Bot 2011-02-14 05:14:07 PST
All reviewed patches have been landed.  Closing bug.