RESOLVED FIXED Bug 69372
[CSS3 Regions] Make sure overflow:visible lets content spill out of regions. 
https://bugs.webkit.org/show_bug.cgi?id=69372
Summary [CSS3 Regions] Make sure overflow:visible lets content spill out of regions.
Dave Hyatt
Reported 2011-10-04 13:49:16 PDT
Make sure overflow:visible lets content spill out of regions. Also make sure regions do not clip their content in the flow thread's inline direction.
Attachments
Patch (579.35 KB, patch)
2011-10-04 13:53 PDT, Dave Hyatt 		no flags
DetailsFormatted DiffDiff
Patch (579.65 KB, patch)
2011-10-04 13:56 PDT, Dave Hyatt 		no flags
DetailsFormatted DiffDiff
Patch (590.59 KB, patch)
2011-10-04 14:05 PDT, Dave Hyatt 		andersca: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Dave Hyatt
Comment 1 2011-10-04 13:53:01 PDT
Created attachment 109677 [details] Patch
WebKit Review Bot
Comment 2 2011-10-04 13:55:51 PDT
Attachment 109677 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/plat..." exit_code: 1 Source/WebCore/rendering/RenderRegion.cpp:129: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] LayoutTests/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] [5] Source/WebCore/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] [5] Total errors found: 3 in 6 files If any of these errors are false positives, please file a bug against check-webkit-style.
Dave Hyatt
Comment 3 2011-10-04 13:56:40 PDT
Created attachment 109680 [details] Patch
WebKit Review Bot
Comment 4 2011-10-04 13:58:53 PDT
Attachment 109680 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/plat..." exit_code: 1 LayoutTests/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] [5] Source/WebCore/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] [5] Total errors found: 2 in 6 files If any of these errors are false positives, please file a bug against check-webkit-style.
Dave Hyatt
Comment 5 2011-10-04 14:05:46 PDT
Created attachment 109684 [details] Patch
WebKit Review Bot
Comment 6 2011-10-04 14:10:59 PDT
Attachment 109684 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/plat..." exit_code: 1 Source/JavaScriptCore/wtf/ListHashSet.h:410: const_reverse_iterator is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Source/JavaScriptCore/wtf/ListHashSet.h:451: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] Total errors found: 2 in 8 files If any of these errors are false positives, please file a bug against check-webkit-style.
Csaba Osztrogonác
Comment 7 2011-10-05 00:03:37 PDT
Patch landed http://trac.webkit.org/changeset/96643, but unfortunately WebKit2 doesn't build on Qt, because of -Werror: cc1plus: warnings being treated as errors In file included from /ramdisk/qt-linux-32-release-webkit2/build/Source/WebCore/rendering/HitTestResult.h:27, from ../include/WebCore/HitTestResult.h:1, from /ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/Shared/WebHitTestResult.h:24, from /ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/UIProcess/WebPageProxy.h:49, from /ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/UIProcess/WebBackForwardList.h:32, from /ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/Shared/SessionState.h:29, from /ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/Shared/WebPageCreationParameters.h:30, from /ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/Shared/WebPageCreationParameters.cpp:27: /ramdisk/qt-linux-32-release-webkit2/build/Source/JavaScriptCore/wtf/ListHashSet.h:429: error: class ‘WTF::ListHashSetConstReverseIterator<ValueArg, inlineCapacity, HashArg>’ is implicitly friends with itself
Csaba Osztrogonác
Comment 8 2011-10-06 05:45:06 PDT
Patch landed in http://trac.webkit.org/changeset/96643 and buildfix laned in http://trac.webkit.org/changeset/96648.
Note You need to log in before you can comment on or make changes to this bug.