Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(727)

Issue 10090001: Merge 112582 - REGRESSION (r110065-r110080): Content drawing outside overflow: hidden at ynet.co.il (Closed)

Created:
8 years, 8 months ago by Julien - ping for review
Modified:
8 years, 8 months ago
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1084/
Visibility:
Public.

Description

Merge 112582 - REGRESSION (r110065-r110080): Content drawing outside overflow: hidden at ynet.co.il https://bugs.webkit.org/show_bug.cgi?id=82129 Reviewed by Ojan Vafai. Source/WebCore: Tests: fast/overflow/before-after-overflow-hidden-horizontal-writing-mode-tb-expected.html fast/overflow/before-after-overflow-hidden-horizontal-writing-mode-tb.html fast/overflow/before-after-overflow-hidden-vertical-writing-mode-rl-expected.html fast/overflow/before-after-overflow-hidden-vertical-writing-mode-rl.html fast/overflow/start-end-overflow-hidden-horizontal-writing-mode-tb-expected.html fast/overflow/start-end-overflow-hidden-horizontal-writing-mode-tb.html fast/overflow/start-end-overflow-hidden-vertical-writing-mode-rl-expected.html fast/overflow/start-end-overflow-hidden-vertical-writing-mode-rl.html This is a regression from r110072. I wrongly thought we should call ensureLayer if we create our RenderOverflow. However the current overflow code removes the before and start overflows (like in horizontal writing mode with ltr direction, we never have a top or a left overflow). Because of that we would not get a RenderLayer as expected and the overflow clip rects would be wrong on our RenderLayer children. * rendering/RenderBox.cpp: (WebCore::RenderBox::addLayoutOverflow): Moved the ensureLayer() call after the check that we do have some overflow but before we remove the overflow in some directions. LayoutTests: * fast/overflow/before-after-overflow-hidden-horizontal-writing-mode-tb-expected.html: Added. * fast/overflow/before-after-overflow-hidden-horizontal-writing-mode-tb.html: Added. * fast/overflow/before-after-overflow-hidden-vertical-writing-mode-rl-expected.html: Added. * fast/overflow/before-after-overflow-hidden-vertical-writing-mode-rl.html: Added. * fast/overflow/start-end-overflow-hidden-horizontal-writing-mode-tb-expected.html: Added. * fast/overflow/start-end-overflow-hidden-horizontal-writing-mode-tb.html: Added. * fast/overflow/start-end-overflow-hidden-vertical-writing-mode-rl-expected.html: Added. * fast/overflow/start-end-overflow-hidden-vertical-writing-mode-rl.html: Added. Added 4 ref tests that should cover the 4 different values of (hasTopOverflow, hasLeftOverflow) in RenderBox::addLayoutOverflow. * platform/chromium-linux/fast/box-shadow/shadow-buffer-partial-expected.txt: * platform/chromium-win/fast/block/lineboxcontain/block-font-expected.txt: * platform/chromium-win/fast/block/lineboxcontain/block-glyphs-expected.txt: * platform/chromium-win/fast/block/lineboxcontain/font-expected.txt: Rebaselined those tests on Chromium linux (new layers). * platform/chromium/test_expectations.txt: * platform/efl/Skipped: * platform/gtk/Skipped: * platform/mac/Skipped: * platform/qt/Skipped: * platform/win/Skipped: * platform/wk2/Skipped: Skipped the previous tests on the other platforms. TBR=jchaffraix@webkit.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=114145

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+78 lines, -31 lines) Patch
A + LayoutTests/fast/overflow/before-after-overflow-hidden-horizontal-writing-mode-tb.html View 0 chunks +-1 lines, --1 lines 0 comments Download
A + LayoutTests/fast/overflow/before-after-overflow-hidden-horizontal-writing-mode-tb-expected.html View 0 chunks +-1 lines, --1 lines 0 comments Download
A + LayoutTests/fast/overflow/before-after-overflow-hidden-vertical-writing-mode-rl.html View 0 chunks +-1 lines, --1 lines 0 comments Download
A + LayoutTests/fast/overflow/before-after-overflow-hidden-vertical-writing-mode-rl-expected.html View 0 chunks +-1 lines, --1 lines 0 comments Download
A + LayoutTests/fast/overflow/start-end-overflow-hidden-horizontal-writing-mode-tb.html View 0 chunks +-1 lines, --1 lines 0 comments Download
A + LayoutTests/fast/overflow/start-end-overflow-hidden-horizontal-writing-mode-tb-expected.html View 0 chunks +-1 lines, --1 lines 0 comments Download
A + LayoutTests/fast/overflow/start-end-overflow-hidden-vertical-writing-mode-rl.html View 0 chunks +-1 lines, --1 lines 0 comments Download
A + LayoutTests/fast/overflow/start-end-overflow-hidden-vertical-writing-mode-rl-expected.html View 0 chunks +-1 lines, --1 lines 0 comments Download
M LayoutTests/platform/chromium-linux/fast/box-shadow/shadow-buffer-partial-expected.txt View 1 chunk +3 lines, -2 lines 0 comments Download
M LayoutTests/platform/chromium-win/fast/block/lineboxcontain/block-font-expected.txt View 1 chunk +12 lines, -11 lines 0 comments Download
M LayoutTests/platform/chromium-win/fast/block/lineboxcontain/block-glyphs-expected.txt View 1 chunk +12 lines, -11 lines 0 comments Download
M LayoutTests/platform/chromium-win/fast/block/lineboxcontain/font-expected.txt View 1 chunk +12 lines, -11 lines 0 comments Download
M LayoutTests/platform/chromium/test_expectations.txt View 1 chunk +6 lines, -0 lines 0 comments Download
M LayoutTests/platform/efl/Skipped View 1 chunk +7 lines, -0 lines 0 comments Download
M LayoutTests/platform/gtk/Skipped View 1 chunk +7 lines, -0 lines 0 comments Download
M LayoutTests/platform/qt/Skipped View 1 chunk +7 lines, -0 lines 0 comments Download
M LayoutTests/platform/win/Skipped View 1 chunk +7 lines, -0 lines 0 comments Download
M LayoutTests/platform/wk2/Skipped View 1 chunk +7 lines, -0 lines 0 comments Download
M Source/WebCore/rendering/RenderBox.cpp View 2 chunks +6 lines, -4 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Julien - ping for review
8 years, 8 months ago (2012-04-13 18:02:24 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698